Skip to content
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.

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

Add option for additional logo in footer #3227

Conversation

precious-onyenaucheya-ons
Copy link
Contributor

@precious-onyenaucheya-ons precious-onyenaucheya-ons commented Jun 4, 2024

What is the context of this PR?

Fixes #3134
This PR updates the footer component to support an additional logo, allowing for two logos instead of just one. It introduces link param for each logo and offers an alternative layout option to display the logos on opposite sides of the footer. The default layout positions the logos next to each other. Additionally, the implementation is aligned with the header component's multipleLogo feature to ensure consistency across the application. The focus and hover states are also adjusted to avoid extra underlines and to ensure the highlight color wraps only the SVG.

BREAKING CHANGE
Services using the poweredBy param with the footer component will now need to update that param to footerLogo object in their config.

How to review this PR

  • Spin up the website and head to the 'footer' component, the example-footer-with-extra-poweredby-logo on the page is the one to look at.
  • Check for multiple logos: Verify that the footer displays two logos as expected, with appropriate links set for each logo.
  • Review layout options: Confirm that the default layout positions the logos next to each other and test the alternative layout option where the logos are displayed on opposite sides of the footer.
  • Test focus and hover states: Ensure that the focus and hover states do not create an extra underline and that the highlight color wraps only the SVG, not its margin.
  • run yarn test --testNamePattern="macro: footer" & yarn test-visual

Checklist

This needs to be completed by the person raising the PR.

  • I have selected the correct Assignee
  • I have linked the correct Issue

precious-onyenaucheya-ons and others added 30 commits April 22, 2024 15:44
@alessioventuriniAND alessioventuriniAND changed the base branch from main to major-release September 3, 2024 09:48
@alessioventuriniAND alessioventuriniAND merged commit b7b1a5b into major-release Sep 5, 2024
7 of 9 checks passed
@alessioventuriniAND alessioventuriniAND deleted the feature/3134-add-option-for-additional-logo-in-footer branch September 5, 2024 13:52
@alessioventuriniAND alessioventuriniAND removed the Do not merge Don't merge this PR until this label is removed label Sep 6, 2024
This was referenced Sep 10, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
rmccar pushed a commit that referenced this pull request Oct 9, 2024
* Standardise setting heading level across components #3111

* Update param names to camel case naming standard #3188

* Renamed utility class from ons-grid--flex to ons-grid-flex #3268

* Removed metadata component #3275

* Remove call to action component and all references #3143

* Update parameter from UnreadLink to UnreadLinkUrl in Message component #3290

* Update parameter subject from text to object type in Message list component #3291

* Add option for additional logo in footer #3227

* Update parameter from link to text in Footer Component #3294

* Update parameter name from url to src in Image Component #3293

* Update parameter names from LogoImage and LogoURL to image and url a in header component #3292

* Update parameter name from url to linkUrl in feedback component #3295

* Rename summary component parameters #3224

* Refactor timeline macro for compatibility with Jinja2 templates #3180

* Update parameter names in document list component #3299

* Update param name settingsLinkTextURL to settingsLinkURL in Cookies Banner component #3300

* Update parameter names in card component #3303

* Update parameter from linkText to text in external link component #3304

* Update param name from title to text in section navigation component #3305

* Update parameter from linkText to videoLinkText in Video Component #3306

* Update parameter from manualLink to manualLinkUrl in Address Input component #3307

* Ensure list item custom attributes are always set on list item #3336

* Rename items summary param for compatibility with Jinja #3382

* Update DS components to ensure compatibility with Jinja #3380

* Rename html OGL Link param to current naming standard #3379

* Format migration guide #3377

* Introduction of new spacing grid #3285
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaking changes This PR contains at least one breaking change Enhancement Change of existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add option for additional logo in footer
6 participants