From 3abd0e8e6230e051e70a6369bb92ade3ddf32810 Mon Sep 17 00:00:00 2001
From: Ned Batchelder <ned@nedbatchelder.com>
Date: Wed, 16 Dec 2020 15:30:55 -0500
Subject: [PATCH] Update the Open edX logo urls (#25898)
The old URLs now serve the new logo, but at the old pixel size. These
URLs serve the new logo at a natural pixel size.
---
cms/templates/widgets/footer.html | 2 +-
lms/djangoapps/branding/tests/test_api.py | 2 +-
lms/envs/common.py | 8 ++++----
lms/templates/footer.html | 4 ++--
themes/edx.org/lms/templates/footer.html | 2 +-
themes/red-theme/lms/templates/footer.html | 8 +++-----
6 files changed, 12 insertions(+), 14 deletions(-)
diff --git a/cms/templates/widgets/footer.html b/cms/templates/widgets/footer.html
index 2afbcbc6cb6..f91e238f328 100644
--- a/cms/templates/widgets/footer.html
+++ b/cms/templates/widgets/footer.html
@@ -56,7 +56,7 @@ from openedx.core.djangolib.markup import HTML, Text
<div class="footer-about-openedx">
<a href="https://open.edx.org" title="${_("Powered by Open edX")}">
- <img alt="${_("Powered by Open edX")}" src="https://files.edx.org/openedx-logos/edx-openedx-logo-tag.png">
+ <img alt="${_("Powered by Open edX")}" src="https://files.edx.org/openedx-logos/open-edx-logo-tag.png">
</a>
</div>
</div>
diff --git a/lms/djangoapps/branding/tests/test_api.py b/lms/djangoapps/branding/tests/test_api.py
index f6f0696b2b6..f832edd0f3c 100644
--- a/lms/djangoapps/branding/tests/test_api.py
+++ b/lms/djangoapps/branding/tests/test_api.py
@@ -157,7 +157,7 @@ class TestFooter(TestCase):
'logo_image': 'https://edx.org/static/images/logo.png',
'openedx_link': {
'url': 'https://open.edx.org',
- 'image': 'https://files.edx.org/openedx-logos/edx-openedx-logo-tag.png',
+ 'image': 'https://files.edx.org/openedx-logos/open-edx-logo-tag.png',
'title': 'Powered by Open edX'
},
'edx_org_link': {
diff --git a/lms/envs/common.py b/lms/envs/common.py
index dc6c7506a04..c5e00559d66 100644
--- a/lms/envs/common.py
+++ b/lms/envs/common.py
@@ -1641,10 +1641,10 @@ FOOTER_OPENEDX_URL = "https://open.edx.org"
# We use logo images served from files.edx.org so we can (roughly) track
# how many OpenEdX installations are running.
# Site operators can choose from these logo options:
-# * https://files.edx.org/openedx-logos/edx-openedx-logo-tag.png
-# * https://files.edx.org/openedx-logos/edx-openedx-logo-tag-light.png"
-# * https://files.edx.org/openedx-logos/edx-openedx-logo-tag-dark.png
-FOOTER_OPENEDX_LOGO_IMAGE = "https://files.edx.org/openedx-logos/edx-openedx-logo-tag.png"
+# * https://files.edx.org/openedx-logos/open-edx-logo-tag.png
+# * https://files.edx.org/openedx-logos/open-edx-logo-tag-light.png"
+# * https://files.edx.org/openedx-logos/open-edx-logo-tag-dark.png
+FOOTER_OPENEDX_LOGO_IMAGE = "https://files.edx.org/openedx-logos/open-edx-logo-tag.png"
# This is just a placeholder image.
# Site operators can customize this with their organization's image.
diff --git a/lms/templates/footer.html b/lms/templates/footer.html
index 37bc3498ce9..a919d6ef533 100644
--- a/lms/templates/footer.html
+++ b/lms/templates/footer.html
@@ -72,7 +72,7 @@
<div class="footer-about-openedx">
<p>
<a href="${footer['openedx_link']['url']}">
- <img src="${footer['openedx_link']['image']}" alt="${footer['openedx_link']['title']}" width="140" />
+ <img src="${footer['openedx_link']['image']}" alt="${footer['openedx_link']['title']}" width="175" />
</a>
</p>
</div>
@@ -151,7 +151,7 @@
<div class="footer-about-openedx">
<p>
<a href="${footer['openedx_link']['url']}">
- <img src="${footer['openedx_link']['image']}" alt="${footer['openedx_link']['title']}" width="140" />
+ <img src="${footer['openedx_link']['image']}" alt="${footer['openedx_link']['title']}" width="175" />
</a>
</p>
</div>
diff --git a/themes/edx.org/lms/templates/footer.html b/themes/edx.org/lms/templates/footer.html
index b2404d9c664..9d54b21076b 100755
--- a/themes/edx.org/lms/templates/footer.html
+++ b/themes/edx.org/lms/templates/footer.html
@@ -95,7 +95,7 @@
% if not hide_openedx_link and hide_openedx_link != Undefined:
<div class="openedx-link">
<a href="${footer['openedx_link']['url']}" title="${footer['openedx_link']['title']}">
- <img alt="${footer['openedx_link']['title']}" src="${footer['openedx_link']['image']}" width="140">
+ <img alt="${footer['openedx_link']['title']}" src="${footer['openedx_link']['image']}" width="175">
</a>
</div>
% endif
diff --git a/themes/red-theme/lms/templates/footer.html b/themes/red-theme/lms/templates/footer.html
index 24ebd2483d9..b126cc976af 100755
--- a/themes/red-theme/lms/templates/footer.html
+++ b/themes/red-theme/lms/templates/footer.html
@@ -110,11 +110,9 @@ from openedx.core.djangolib.markup import HTML, Text
<a href="http://openedx.org/">
## standard powered-by logo
## Translators: 'Open edX' is a brand, please keep this untranslated. See http://openedx.org for more information.
- <img src="https://files.edx.org/openedx-logos/edx-openedx-logo-tag.png" alt="${_('Powered by Open edX')}" width="140" />
- ## greyscale logo for dark background
- ## <img src="https://files.edx.org/openedx-logos/edx-openedx-logo-tag-light.png" alt="${_('Powered by Open edX')}" width="140" />
- ## greyscale logo for light background
- ## <img src="https://files.edx.org/openedx-logos/edx-openedx-logo-tag-dark.png" alt="${_('Powered by Open edX')}" width="140" />
+ <img src="https://files.edx.org/openedx-logos/open-edx-logo-tag.png" alt="${_('Powered by Open edX')}" width="175" />
+ ## white logo for dark background
+ ## <img src="https://files.edx.org/openedx-logos/open-edx-logo-tag-dark.png" alt="${_('Powered by Open edX')}" width="175" />
</a>
</p>
</div>
--
GitLab