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

fix: material icons font #89

Closed
wants to merge 1 commit into from
Closed

Conversation

HenryQW
Copy link

@HenryQW HenryQW commented Mar 2, 2021

material icons font is installed through composer now, this PR updates all the paths.

https://git.tt-rss.org/fox/tt-rss/commit/a22ddb2fe0d538ef930a47d480dd71ca1ad9f8d4

Close HenryQW/Awesome-TTRSS#273.

@eric-pierce
Copy link

Thanks for addressing @HenryQW!

@levito
Copy link
Owner

levito commented Mar 4, 2021

Hey, thanks for the PR! Unfortunately, I can't accept it like that since you edited generated files. The proper way to address this would be updating the tt-rss submodule and rebuilding the styles.

I just published a new release which is compatible with the recent tt-rss changes: https://github.com/levito/tt-rss-feedly-theme/releases/tag/v2.7.0

Cheers!

@levito levito closed this Mar 4, 2021
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.

[BUG] Material-icons moved to composer, missing on Awesome-TTRSS
3 participants