-
Notifications
You must be signed in to change notification settings - Fork 228
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
Theme parsing error: gtk-dark.css:9381:21: Not using units is deprecated. Assuming 'px'. #757
Comments
Hi, just wanted to follow up if there was any chance to look into this? |
hi, |
after some research, it seems that the problem comes from using cd /usr/share/themes/Numix/gtk-3.20/
gresource extract gtk.gresource /org/numixproject/gtk-3.20/dist/gtk.css > gtk.css
sed -i 's/border-top-width: 1;$/border-top-width: 1px;/' gtk.css in fact, the error seems to be linked to this single line which is the only one to contain the string
I hope this help for the dark theme, regards, lacsaP. |
Extracting those does not even result in files having enough lines (e.g. 9381). Also tried to search for the property you linked and does not seem to be there. |
after extracting and replacing cd /usr/share/themes/Numix/gtk-3.20/
egrep -n '[1-9]; gtk-dark.css | grep -v '#' not all lines are concerned ( regards, lacsaP. |
the string the use of the previous sed -i 's/border-top-width: 1;$/border-top-width: 1px;/' gtk-dark.css regards, lacsaP. |
@patatetom, man, thanks a lot. I realized that on my first attempt I tampered with an outdated |
@darkshram, it comes from #751, where @Ferdi265 already pointed this out. Merged by @khurshid-alam. |
It would be great if someone make a merge request. This is Saas compiler issue where it is treating this as error instead of warning. Fixed in debian I think. |
@Foggalong, why closed? |
@Alexander-Shukaev My bad, I misread your last message as saying #751 had closed it! |
…s deprecated. Assuming 'px'.
Moved from Ferdi265#29.
Hi there, I see a lot of warnings after system upgrade:
I think some styles need to be updated to include
px
where it's meant. Thank you!The text was updated successfully, but these errors were encountered: