Skip to content

Fix theme parser errors #170

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

siddhpant
Copy link

Thanks to @nullorigin for fixing them in his fork, but he replaced in files directly. In this PR I fix it in the generator files and regenerate the themes.

Fixes the following warnings:
	Theme parser error: gtk.css:5050:12-15: "mix" is not a valid color name.
	Theme parser error: gtk.css:5054:12-15: "mix" is not a valid color name.

Apparently there should be no space between function name and the
opening parentheses.

The fix was originally done by Matthew Fay in his fork:
https://github.com/nullorigin/flat-remix-gtk

We will fix the main file and regenerate the themes later.

Suggested-by: Matthew Fay <[email protected]>
Signed-off-by: Siddh Raman Pant <[email protected]>
Fixes the following warnings:
	Theme parser error: gtk.css:4963:12-17: "shade" is not a valid color name.
	Theme parser error: gtk.css:5023:12-17: "shade" is not a valid color name.

Apparently there should be no space between function name and the
opening parentheses.

The fix was originally done by Matthew Fay in his fork:
https://github.com/nullorigin/flat-remix-gtk

We will fix the main files and regenerate the themes later.

Suggested-by: Matthew Fay <[email protected]>
Signed-off-by: Siddh Raman Pant <[email protected]>
@nullorigin
Copy link

nullorigin commented Apr 10, 2025 via email

@siddhpant siddhpant force-pushed the fix-parser-errors branch 2 times, most recently from 1b460da to beb963b Compare April 14, 2025 09:25
Fixes the following warning:
	Theme parser error: gtk.css:5286:3-19: No property named "-gtk-icon-effect"

Let's complete the TODO mentioned.

The fix was originally done by Matthew Fay in his fork:
https://github.com/nullorigin/flat-remix-gtk

We will be more exact, and fix the main file by doing a sed hack in the
sass Makefile, and regenerate the themes later.

Suggested-by: Matthew Fay <[email protected]>
Signed-off-by: Siddh Raman Pant <[email protected]>
Fixes the following warning:
        Theme parser warning: gtk.css:5255:3-5256:1: Expected ';' at end of block

We will fix the main file and regenerate the themes later.

Signed-off-by: Siddh Raman Pant <[email protected]>
Signed-off-by: Siddh Raman Pant <[email protected]>
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.

2 participants