Summary: | Web Inspector: -webkit-text-size-adjust should be added to the completion list | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Timothy Hatcher <timothy> | ||||
Component: | Web Inspector | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | graouts, hi, inspector-bugzilla-changes, mmaxfield, simon.fraser, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Local Build | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
CSS.getSupportedCSSProperties only returns the list of supported properties by the platform you are inspecting. OS X does not support that property, only iOS. So when inspecting a page on OS X this will probably happen, but inspecting on iOS it won't. Should we have a list of fuzzy properties to not warn about no matter if the backend supports it? This would likely include any iOS / well known other vendor properties. (In reply to comment #2) > CSS.getSupportedCSSProperties only returns the list of supported properties > by the platform you are inspecting. OS X does not support that property, > only iOS. I suppose OS X could support the property and just do nothing for it. "none" and "auto" would both mean the same thing. Myles / Simon what do you think about this, is it a bad idea? Yeah, I was mainly thinking about making the warning part not care about iOS. It already ignores properties with other vendor prefixes. (In reply to comment #3) > (In reply to comment #2) > > CSS.getSupportedCSSProperties only returns the list of supported properties > > by the platform you are inspecting. OS X does not support that property, > > only iOS. > > I suppose OS X could support the property and just do nothing for it. "none" > and "auto" would both mean the same thing. > > Myles / Simon what do you think about this, is it a bad idea? I think this would be a fine idea. Having it work on Mac conditionally would be helpful too. Then we could enable it for responsive design mode. This works as of r214067 <https://trac.webkit.org/changeset/214067>. |
Created attachment 266530 [details] Screenshot We should know about -webkit-text-size-adjust, so we don't show a warning icon for it.