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

Update Footer "Follow Us" Logo from Twitter to X #11170

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Pawan8538
Copy link

@Pawan8538 Pawan8538 commented Dec 24, 2024


What:
This PR updates the footer "Follow Us" logo on the OpenFoodFacts website to reflect the recent rebranding of Twitter to X. The old Twitter logo has been replaced with the new X logo, ensuring that the footer is aligned with the current branding of the platform.

Changes:

  • Replaced the Twitter logo in the footer with the new X logo.
  • Updated the link to point to https://x.com/openfoodfacts instead of https://twitter.com/openfoodfacts.
  • common.pot and en.po files changed too.

How to Test:

  1. Go to any page on the OpenFoodFacts website (such as the homepage or product page).
  2. Scroll down to the footer and locate the "Follow Us" section.
  3. Verify that the X logo is now visible instead of the old Twitter logo.
  4. Ensure that clicking the X logo redirects to the OpenFoodFacts X profile: https://x.com/openfoodfacts.
  5. Check that the footer displays correctly on both desktop and mobile devices.

@Pawan8538 Pawan8538 requested a review from a team as a code owner December 24, 2024 10:08
@github-actions github-actions bot added Template::Toolkit The templating toolkit used by product opener. The starting point for HTML/JS/CSS fixes. icons 🌐 Translations Site layout labels Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
icons Site layout Template::Toolkit The templating toolkit used by product opener. The starting point for HTML/JS/CSS fixes. 🌐 Translations
Development

Successfully merging this pull request may close these issues.

2 participants