Skip to content
This repository has been archived by the owner on May 29, 2018. It is now read-only.

Add GTK-3.22 theme #414

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add GTK-3.22 theme #414

wants to merge 1 commit into from

Conversation

Narrat
Copy link

@Narrat Narrat commented Oct 18, 2016

Based upon the 3.20 theme. It was only adjusted in those parts, where GTK-3.22 complained.

Mainly:

Theme parsing error: gtk.css:6363:24: Using Pango syntax for the font: style property is deprecated; please use CSS syntax
--> Font: Changing font to font-family

Theme parsing error: gtk.css:453:56: Using one color stop with linear-gradient() is deprecated.
--> linear-gradient(): Adding 'transparent' where the second color stop argument were missing (_entries.scss 124,128; _scales.scss: 368; _drawing.scss: 71, 77, 83, 89, 98, 107, 116, 304, 377, 392, 406, 418, 433, 456; _header-bars: 68)

Theme parsing error: gtk.css:6936:10: Not using units is deprecated.Assuming 'px'.
--> px added

Remaing issues:

  • Theme parsing error: gtk.css:5446:11: Expected a string.
  • Theme parsing error: gtk.css:5673:16: not a number

Regarding specified fonts like "font: 18px;""

Closes #409


Again. Dunno if it's really OK what I have done. I did not notice any drawbacks using the modified theme.

Remaining stuff ware the warning messages regarding "Expected a string" and "not a number". Didn't find information on it. So not sure if changing it to font-family will help, or specifing a dummy font in those cases.

@Narrat
Copy link
Author

Narrat commented Oct 22, 2016

Found a solution for the remaining messages.

  • font followed by a modifier --> font-weight
  • font followed by font size --> font-size

Still unsure about the linear-gradient thing. If transpararent was the right value. I saw other implementations, where the first color stop value is used for the second one.

Based upon the 3.20 theme. It was only adjusted in those parts, where GTK-3.22 complained.

Mainly:

Theme parsing error: gtk.css:6363:24: Using Pango syntax for the font: style property is deprecated; please use CSS syntax
    --> Font: Changing font to font-family

Theme parsing error: gtk.css:453:56: Using one color stop with linear-gradient() is deprecated.
    --> linear-gradient(): Adding 'transparent' where the second color stop argument were missing (_entries.scss 124,128; _scales.scss: 368; _drawing.scss: 71, 77, 83, 89, 98, 107, 116, 304, 377, 392, 406, 418, 433, 456; _header-bars: 68)

Theme parsing error: gtk.css:6936:10: Not using units is deprecated.Assuming 'px'.
    --> px added

Theme parsing error: gtk.css:5446:11: Expected a string.
    --> Changing font to font-size
Theme parsing error: gtk.css:5673:16: not a number
    --> Changing font to font-weight

Closes #409
@Narrat Narrat changed the title Add GTK-3.22 theme [WIP] Add GTK-3.22 theme Oct 22, 2016
@develop7
Copy link

@snwh have you had a chance to review this?

@develop7
Copy link

All right, just have tested it, works just fine, as warnings seem to be gone. Awesome job, @Narrat!

@snwh :shipit:

@lluixhi
Copy link

lluixhi commented Apr 17, 2017

@snwh
Can you merge this?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants