Skip to content

chore: refactor footer and header icons#1962

Merged
bjohansebas merged 25 commits intoexpressjs:gh-pagesfrom
ShubhamOulkar:refactor-footer
Jul 22, 2025
Merged

chore: refactor footer and header icons#1962
bjohansebas merged 25 commits intoexpressjs:gh-pagesfrom
ShubhamOulkar:refactor-footer

Conversation

@ShubhamOulkar
Copy link
Copy Markdown
Member

@ShubhamOulkar ShubhamOulkar commented Jul 5, 2025

  • Implement theme toggle button with modern mask-image
  • Add dynamic theme support using --box-fg for theme toggle button
  • Replace dual logo images with single background-image for OpenJS Foundation logo
  • Implement automatic logo toggle based on .dark-mode
  • Remove Font Awesome dependency
  • Replace icons with optimized local SVG
  • Clean up redundant SVGs (remove duplicates or unused icons)
  • fix bugs and refactor theme.js

~6 KB reduction in CSS payload

image

@ShubhamOulkar ShubhamOulkar requested a review from a team as a code owner July 5, 2025 13:58
@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 5, 2025

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit dd2c457
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/687f011da2624d000965276e
😎 Deploy Preview https://deploy-preview-1962--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@ShubhamOulkar ShubhamOulkar marked this pull request as draft July 5, 2025 13:59
@ShubhamOulkar ShubhamOulkar changed the title refactor footer chore: refactor footer and header icons Jul 5, 2025
@ShubhamOulkar ShubhamOulkar added the maintenance Issues/PRs related to making the website maintainable label Jul 5, 2025
Comment thread _includes/footer.html Outdated
Comment thread _includes/header.html Outdated
@ShubhamOulkar ShubhamOulkar marked this pull request as ready for review July 6, 2025 07:08
Comment thread _includes/footer.html Outdated
Comment thread css/style.css Outdated
Comment thread images/moon.svg Outdated
Comment thread images/sun.svg Outdated
Comment thread _includes/icons/github.svg Outdated
@ShubhamOulkar ShubhamOulkar self-assigned this Jul 19, 2025
@ShubhamOulkar
Copy link
Copy Markdown
Member Author

Hi @bjohansebas, Can you review this? I want to land this early as possible.🙏

Comment thread _includes/icons/hamburger.svg Outdated
@crandmck crandmck self-requested a review July 21, 2025 14:46
Copy link
Copy Markdown
Member

@crandmck crandmck left a comment

Choose a reason for hiding this comment

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

LGTM, but I didn't test it comprehensively.
I trust @bjohansebas and others will do so.

Copy link
Copy Markdown
Member

@bjohansebas bjohansebas left a comment

Choose a reason for hiding this comment

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

The icons are not centered.
imagen
imagen

@ShubhamOulkar
Copy link
Copy Markdown
Member Author

ShubhamOulkar commented Jul 22, 2025

LGTM
image

image image

Copy link
Copy Markdown
Member

@bjohansebas bjohansebas left a comment

Choose a reason for hiding this comment

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

LGTM

@bjohansebas bjohansebas merged commit 1917b49 into expressjs:gh-pages Jul 22, 2025
12 checks passed
@bjohansebas
Copy link
Copy Markdown
Member

thanks @ShubhamOulkar!

@ShubhamOulkar ShubhamOulkar deleted the refactor-footer branch July 23, 2025 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Issues/PRs related to making the website maintainable

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants