Sebastian Zartner [:sebo] Reporter
	Description • 8 years ago

There are several CSS properties, which are missing an auto-completion of their values, like e.g. font-weight.

Sebastian

	Tom Tromey :tromey
	Comment 1 • 8 years ago

Based on looking at nsCSSPropList.h, I think font-weight will be fixed by the patch
in bug 1106336.  So maybe this should just be closed as a dup?

	Sebastian Zartner [:sebo] Reporter
	Comment 2 • 8 years ago

Bug 1106336 doesn't have a patch yet and is restricted to the 'text-overflow' property. This one is meant as meta-bug for all properties.

Sebastian

	Tom Tromey :tromey
	Comment 3 • 8 years ago

Oh, sorry about that!  I confused it with the bug where this is being done, namely bug 1156019.
I think that will fix both 1106336 and this bug.

	Tom Tromey :tromey
	Comment 4 • 7 years ago

I think this bug can probably be closed, but I thought I'd ask first.

	Sebastian Zartner [:sebo] Reporter
	Comment 5 • 7 years ago

While bug 1156019 brought a huge improvement in autocompleting values, there are still a few properties that are missing autocompletion like the 'none' value for 'quotes', for example. I didn't check this in detailed yet, though I'd like to keep this bug open and file more blocking bugs for the missing pieces.

Sebastian

	Tom Tromey :tromey
	Comment 6 • 7 years ago

(In reply to Sebastian Zartner [:sebo] from comment #5)
> While bug 1156019 brought a huge improvement in autocompleting values, there
> are still a few properties that are missing autocompletion like the 'none'
> value for 'quotes', for example.

Thanks!  I wonder how you found this.

"quotes" has a custom parser:

https://dxr.mozilla.org/mozilla-central/rev/f40e24f40b4c4556944c762d4764eace261297f5/layout/style/nsCSSPropList.h#3551

and it recognizes none here:

https://dxr.mozilla.org/mozilla-central/rev/f40e24f40b4c4556944c762d4764eace261297f5/layout/style/nsCSSParser.cpp#15356

Perhaps all `CSS_PROPERTY_PARSE_FUNCTION` properties ought to be audited; though I wonder
if there's some more future-proof way.

	Sebastian Zartner [:sebo] Reporter
	Comment 7 • 7 years ago

(In reply to Tom Tromey :tromey from comment #6)
> (In reply to Sebastian Zartner [:sebo] from comment #5)
> > While bug 1156019 brought a huge improvement in autocompleting values, there
> > are still a few properties that are missing autocompletion like the 'none'
> > value for 'quotes', for example.
> 
> Thanks!  I wonder how you found this.

In this case simply by trying out the different properties in the DevTools.

> Perhaps all `CSS_PROPERTY_PARSE_FUNCTION` properties ought to be audited;

For now, I think there's no other way to do it.

> though I wonder if there's some more future-proof way.

Maybe the related code could be changed to somehow expose their possible values right in the related parse functions, though I don't really know the code enough to give proper input on this.

Sebastian