Skip to content

Conversation

@MirruK
Copy link
Contributor

@MirruK MirruK commented Jan 23, 2026

Add styles for fontawesome icons to respect hidden and invisible classes

Description

Icons were removed from layers, this caused the .hidden class to not have any effect on icons, this fixes the problem globally

Copilot AI review requested due to automatic review settings January 23, 2026 20:00
@monkeytypegeorge monkeytypegeorge added the frontend User interface or web stuff label Jan 23, 2026
@github-actions github-actions bot added the waiting for review Pull requests that require a review before continuing label Jan 23, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds CSS styles to ensure FontAwesome icons respect the .hidden and .invisible utility classes. The issue arose because FontAwesome icons were removed from CSS layers (due to screenshotting library compatibility issues), which prevented the layered .hidden class from affecting icons.

Changes:

  • Added explicit styles for FontAwesome icon classes (.fas, .fab, .fa, .far) combined with .hidden to set display: none
  • Added explicit styles for FontAwesome icon classes combined with .invisible to set opacity: 0, pointer-events: none, and visibility: hidden

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MirruK MirruK changed the title fontawesome icon visibility styles fix: fontawesome icon visibility styles (@MirruK) Jan 23, 2026
Add styles for fontawesome icons to respect hidden and invisible classes
@Miodec
Copy link
Member

Miodec commented Jan 23, 2026

Thanks

@Miodec Miodec merged commit 9a76dbd into monkeytypegame:master Jan 23, 2026
13 checks passed
@github-actions github-actions bot removed the waiting for review Pull requests that require a review before continuing label Jan 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend User interface or web stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants