Skip to content

SE-3690 Update the Open edX logo urls (backport to opencraft-release/juniper.3) #299

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
samuelallan72 merged 1 commit into from
Dec 19, 2020
Merged

SE-3690 Update the Open edX logo urls (backport to opencraft-release/juniper.3) #299

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions cms/templates/widgets/footer.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,8 +55,8 @@
</div>

<div class="footer-about-openedx">
<a href="http://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">
<a href="https://open.edx.org" title="${_("Powered by Open edX")}">
<img alt="${_("Powered by Open edX")}" src="https://files.edx.org/openedx-logos/open-edx-logo-tag.png">
</a>
</div>
</div>
Expand Down
4 changes: 2 additions & 2 deletions lms/djangoapps/branding/tests/test_api.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -156,8 +156,8 @@ def test_get_footer(self):
'mobile_links': [],
'logo_image': 'https://edx.org/static/images/logo.png',
'openedx_link': {
'url': 'http://open.edx.org',
'image': 'https://files.edx.org/openedx-logos/edx-openedx-logo-tag.png',
'url': 'https://open.edx.org',
'image': 'https://files.edx.org/openedx-logos/open-edx-logo-tag.png',
'title': 'Powered by Open edX'
},
'edx_org_link': {
Expand Down
8 changes: 4 additions & 4 deletions lms/envs/common.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1436,10 +1436,10 @@ def _make_locale_paths(settings): # pylint: disable=missing-function-docstring
# 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.
Expand Down
4 changes: 2 additions & 2 deletions lms/templates/footer.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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>
Expand Down Expand Up @@ -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>
Expand Down
2 changes: 1 addition & 1 deletion themes/edx.org/lms/templates/footer.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ <h2 class="heading">${_(u"Connect")}</h2>
% 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
Expand Down
8 changes: 3 additions & 5 deletions themes/red-theme/lms/templates/footer.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -110,11 +110,9 @@
<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>
Expand Down