Skip to content

Conversation

@omerbakr
Copy link

@omerbakr omerbakr commented Feb 4, 2026

Description

This PR updates the footer to include the required OpenJS Foundation trademark notice as requested in issue #2172.

Changes Made

  • Added the trademark legal text to _data/en.yml.
  • Updated _includes/footer.html to display the text.
  • Added a new CSS rule #footer-trademark-legal in css/style.css to style the text (opacity, font-size) consistent with the footer design.

⚠️ Note on Localization (Important)

Since this is a legal trademark notice, I assumed it should remain in English for all locales unless an official translation is provided.

Instead of duplicating the English text across all language files (es.yml, fr.yml, etc.) which would violate DRY principles, I implemented a Liquid fallback in the footer template:

{{ site.data[page.lang].footer.trademark_legal | default: site.data.en.footer.trademark_legal }}

Signed-off-by: omerbakr <y.omerbakir@gmail.com>
@omerbakr omerbakr requested a review from a team as a code owner February 4, 2026 00:49
@netlify
Copy link

netlify bot commented Feb 4, 2026

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit d768af2
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/6982979a31469400094c27da
😎 Deploy Preview https://deploy-preview-2178--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 96 (🔴 down 2 from production)
Accessibility: 87 (no change from production)
Best Practices: 100 (no change from production)
SEO: 94 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

<a id="coc" href="https://github.com/expressjs/.github/blob/HEAD/CODE_OF_CONDUCT.md">
{{ site.data[page.lang].footer.coc }}
</a>
<a href="https://trademark-policy.openjsf.org">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we remove this duplicated trademark policy link?

cc @expressjs/docs-captains

@ShubhamOulkar
Copy link
Member

Thank you @omerbakr for your contribution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants