Skip to content

feat: add navigation links to make footer consistent #13

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
ahmed-arb merged 17 commits into from
Aug 29, 2025
Merged

feat: add navigation links to make footer consistent #13

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
4e083fd
fix: made footer consistent where using custom footer
May 27, 2025
84988dc
fix: added LMS footer navlinks
May 29, 2025
8393f68
fix: linting error fixed and removed unecessary vars
Jun 6, 2025
eb2cf3d
fix: code refactoring
Jun 6, 2025
0ce52e3
fix: code refactoring
Jun 6, 2025
d6f6833
fix: code clean
Jun 6, 2025
b764c82
fix: snapshot updated
Jun 10, 2025
40fb466
Merge branch 'teak/indigo' into zubair/TIA-137-61
zubairshakoorarbisoft Jun 10, 2025
973e1b8
fix: redundant images removed after conflict resolution
Jun 10, 2025
e3ed25f
fix: conflict resolved and snapshot updated
Jun 10, 2025
a8ee834
fix: updated footer links according legacy pages
Jun 18, 2025
fe2cf7a
fix: code ckean
Jun 18, 2025
a339ec0
fix: code clean
Jun 18, 2025
c14be31
fix: code clean
Jun 18, 2025
c5d04cb
fix: lint errors fixed
Jun 18, 2025
4f6b37d
fix: fixed typo
Jun 19, 2025
4e78bcc
test: update the test coverage for the footer nav
arbirali Jun 23, 2025
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
11 changes: 11 additions & 0 deletions src/components/Footer.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ import LanguageSelector from './LanguageSelector';
ensureConfig([
'LMS_BASE_URL',
'LOGO_TRADEMARK_URL',
'INDIGO_FOOTER_NAV_LINKS',
], 'Footer component');

const EVENT_NAMES = {
Expand Down Expand Up @@ -42,6 +43,7 @@ class SiteFooter extends React.Component {
} = this.props;
const showLanguageSelector = supportedLanguages.length > 0 && onLanguageSelected;
const config = getConfig();
const indigoFooterNavLinks = config.INDIGO_FOOTER_NAV_LINKS || [];

return (
<div className="wrapper wrapper-footer">
Expand Down Expand Up @@ -70,6 +72,15 @@ class SiteFooter extends React.Component {
</li>
</ul>
</div>
<nav className="nav-colophon">
<ol>
{indigoFooterNavLinks.map((link) => (
<li key={link.url}>
<a href={`${config.LMS_BASE_URL}${link.url}`}>{link.title}</a>
</li>
))}
</ol>
</nav>
</div>
<span className="copyright-site">{intl.formatMessage(messages['footer.copyright.text'])}</span>
{showLanguageSelector && (
Expand Down
11 changes: 11 additions & 0 deletions src/components/Footer.test.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,17 @@ import { AppContext } from '@edx/frontend-platform/react';

import Footer from './Footer';

jest.mock('@edx/frontend-platform', () => ({
...jest.requireActual('@edx/frontend-platform'),
getConfig: () => ({
LMS_BASE_URL: process.env.LMS_BASE_URL,
INDIGO_FOOTER_NAV_LINKS: [
{ title: 'About Us', url: '/about' },
{ title: 'Contact', url: '/contact' },
],
}),
}));

const FooterWithContext = ({ locale = 'es' }) => {
const contextValue = useMemo(() => ({
authenticatedUser: null,
Expand Down
60 changes: 60 additions & 0 deletions src/components/__snapshots__/Footer.test.jsx.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,26 @@ exports[`<Footer /> renders correctly renders with a language selector 1`] = `
</li>
</ul>
</div>
<nav
className="nav-colophon"
>
<ol>
<li>
<a
href="http://localhost:18000/about"
>
About Us
</a>
</li>
<li>
<a
href="http://localhost:18000/contact"
>
Contact
</a>
</li>
</ol>
</nav>
</div>
<span
className="copyright-site"
Expand Down Expand Up @@ -145,6 +165,26 @@ exports[`<Footer /> renders correctly renders without a language selector 1`] =
</li>
</ul>
</div>
<nav
className="nav-colophon"
>
<ol>
<li>
<a
href="http://localhost:18000/about"
>
About Us
</a>
</li>
<li>
<a
href="http://localhost:18000/contact"
>
Contact
</a>
</li>
</ol>
</nav>
</div>
<span
className="copyright-site"
Expand Down Expand Up @@ -203,6 +243,26 @@ exports[`<Footer /> renders correctly renders without a language selector in es
</li>
</ul>
</div>
<nav
className="nav-colophon"
>
<ol>
<li>
<a
href="http://localhost:18000/about"
>
About Us
</a>
</li>
<li>
<a
href="http://localhost:18000/contact"
>
Contact
</a>
</li>
</ol>
</nav>
</div>
<span
className="copyright-site"
Expand Down