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

switch from fontawesome to forkawesome #12

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

Conversation

SlidingHorn
Copy link
Contributor

This is completely a judgement call for the waycrate team. Fork Awesome provides a Matrix logo glyph, and is a fully-free, libre and open-source fork of the original FontAwesome, which has apparently privatized its build system.

The drawback at this point is that the "toggle" icons used for the dark mode switch on the website aren't available, so I used a cirlce and a dot circle in their place.

I have not built and run the site locally to see how they show up, so you may want to check that first before merging.

@Shinyzenith
Copy link
Member

I'll wait on angelo for this one. @angelofallars

@SlidingHorn
Copy link
Contributor Author

c47608b corrects an oversight in the CSS - I hadn't previously realized that the classes for the icons are directly referenced in the stylesheet to hide whichever one is currently applied.

@Shinyzenith Shinyzenith force-pushed the main branch 2 times, most recently from 3c9c9e9 to a980b21 Compare June 1, 2022 03:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants