Invalid line-height: inherit error

This forum is for all Flare issues related to styles, stylesheets and XML.

Invalid line-height: inherit error

Postby chuck_agari on Fri Aug 10, 2018 2:52 pm

I'm getting an error now that I have Font Awesome locally: CSS: Invalid line-height: inherit. I looked, and this property-value pair is in nearly every selector in the Font Awesome CSS file.

Actually, it's not quite so much an error as a message. Which opens up the Messages pane pretty much every time I'm working in the file, taking away screen real estate.

According to MDN, inherit is not a valid value for the line-height property. But I could not find anything about this on the W3C site, and CSS Lint did not flag it as an error. Which made me wonder if it's a newly allowed value that Flare has not caught up with yet.

Because it's not an error, I'm assuming that it's just being properly ignored by the parser, which is correct for CSS when there's an invalid value for a property, but the constant messaging is annoying. I've written the Font Awesome folks to get their take before I indulged my initial impulse: to just delete that property-value pair throughout the CSS file. But I'd like to hear from any CSS experts here who can give me some guidance.

thx
chuck_agari
Propeller Head
 
Posts: 45
Joined: Wed May 30, 2018 2:40 pm

Re: Invalid line-height: inherit error

Postby Psider on Mon Aug 13, 2018 6:06 pm

MDN lists it as a global value and Quirksmode says it should work for any property:
https://www.quirksmode.org/css/cascading/values.html

So it's probably a quirk with Flare and you should log a bug?
Psider
Sr. Propeller Head
 
Posts: 431
Joined: Wed Jul 06, 2011 1:32 am

Re: Invalid line-height: inherit error

Postby Dave Lee on Mon Aug 13, 2018 11:58 pm

Try putting the CSS links to font awesome in your master page - I think that should bypass Flare's CSS parser, and stop the error message.
Dave Lee
Master Propellus Maximus
 
Posts: 5265
Joined: Mon Mar 05, 2007 8:33 am
Location: UK

Re: Invalid line-height: inherit error

Postby chuck_agari on Tue Aug 14, 2018 11:29 am

Dave Lee wrote:Try putting the CSS links to font awesome in your master page - I think that should bypass Flare's CSS parser, and stop the error message.

I don't want it in my master page. It clutters up the Style pane with hundreds of styles, making it much harder to find and apply the styles I do use.
chuck_agari
Propeller Head
 
Posts: 45
Joined: Wed May 30, 2018 2:40 pm

Re: Invalid line-height: inherit error

Postby Dave Lee on Thu Aug 16, 2018 12:02 am

chuck_agari wrote:
Dave Lee wrote:Try putting the CSS links to font awesome in your master page - I think that should bypass Flare's CSS parser, and stop the error message.

I don't want it in my master page. It clutters up the Style pane with hundreds of styles, making it much harder to find and apply the styles I do use.


Well, not really - it doesn't show the FA styles when you edit topics.
It only shows FA styles (and CSS parser errors) when you edit the master page itself, which isn't going to be so much of an issue.
Dave Lee
Master Propellus Maximus
 
Posts: 5265
Joined: Mon Mar 05, 2007 8:33 am
Location: UK

Re: Invalid line-height: inherit error

Postby chuck_agari on Thu Aug 23, 2018 2:24 pm

Dave Lee wrote:
chuck_agari wrote:
Dave Lee wrote:Try putting the CSS links to font awesome in your master page - I think that should bypass Flare's CSS parser, and stop the error message.

I don't want it in my master page. It clutters up the Style pane with hundreds of styles, making it much harder to find and apply the styles I do use.


Well, not really - it doesn't show the FA styles when you edit topics.
It only shows FA styles (and CSS parser errors) when you edit the master page itself, which isn't going to be so much of an issue.

Interesting. It appears that you're right, and I'm curious why that is.

Then again, before I created a new project, I hacked the Font Awesome CSS so that it contained only the glyphs I am using. None of them use that property, but if they did, I'd delete it from my hacked version.
chuck_agari
Propeller Head
 
Posts: 45
Joined: Wed May 30, 2018 2:40 pm

Re: Invalid line-height: inherit error

Postby Dave Lee on Fri Aug 24, 2018 12:43 am

chuck_agari wrote:Interesting. It appears that you're right, and I'm curious why that is.


Yeah, the style list for topics will show styles from (a) stylesheets linked directly in that topic, and (b) the master stylesheet (of the current selected primary target).

The list doesn't seem to show styles for stylesheets linked from a master page - which is probably a bug, but means it's a handy way to hide styles from the list.

Flare desperately needs a better way to hide styles, especially when using 3rd party CSS like FontAwesome. You can disable styles in Flare (so they don't appear in the list), but that has serious bugs (doesn't work for any generic class), and isn't a practical method when you need to hide a few thousand styles.

I have to pin all styles that I use. Luckily you can copy the pinned styles list (Project\Users\your.name\PinnedStyles.xml) between users/projects.
Dave Lee
Master Propellus Maximus
 
Posts: 5265
Joined: Mon Mar 05, 2007 8:33 am
Location: UK


Return to Styles, Stylesheets and XML

Who is online

Users browsing this forum: No registered users and 2 guests