Closed Bug 1649021 Opened 4 years ago Closed 4 years ago

[ruleview] Implement the inline CSS compatibility warning in ruleview

Categories

(DevTools :: Inspector: Rules, enhancement, P3)

79 Branch
enhancement

Tracking

(firefox81 fixed)

RESOLVED FIXED
81 Branch
Tracking Status
firefox81 --- fixed

People

(Reporter: lelouch.cpp, Assigned: lelouch.cpp)

References

(Blocks 1 open bug, Regressed 1 open bug)

Details

(Keywords: dev-doc-needed)

Attachments

(3 files, 3 obsolete files)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:79.0) Gecko/20100101 Firefox/79.0

Expected results:

This covers the implementation of the tooltip component for the inline CSS compatibility warning tooltip

Use CompatibilityActor to fetch the compatiblity information for
a particular CSS declaration and show a tooltip for the
incompatible CSS declaration that may cause issue on platform

Depends on D81473

Assignee: nobody → lelouch.cpp
Severity: -- → S3
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Priority: -- → P3
Depends on: 1649020

Given we need the template to be a string, we can use the
react-dom-server to render the UnsupportedBrowserList component
as a string thus reducing code duplication.

Note: This is an RFC and not a definite implementatio path.

Attachment #9163690 - Attachment is obsolete: true
Attached patch use_browserloader.diff (deleted) — Splinter Review

A small patch that reuses the existing UnsupportedBrowserList component by loading the module into the compatibility tooltip helper's own environment.

This patch moves the UnsupportedBrowserList React component and it's dependencies
to devtools/client/shared/compatibility
All the paths linked to the moved components have been updated to
point to the components in shared folder

Depends on D81474

The CSS Compatibility Tooltip tests cover the following cases:

  • Check if the contents of the tooltip match the rendered template
  • Check compatible rules aren't falsely marked incompatible
  • Check incompatible rules are marked correct with icon appearing next to it
  • Check toggling rule disable hides the icon
  • Check adding rules updates the compatibilility status
  • Check the compatibility icon disappears if a fix is added

Depends on D83909

Based on implementation in D49256,this patch moves the snapshot
tests for the components shared between the compatibility panel and the
compatibility tooltip to shared/compatibility/test/node
The components UnsupportedBrowserList and UnsupportedBrowserItem
were moved to devtools/client/shared/compatibility in D83909.
The lock files and configs are copied from
devtools/client/inspector/compatibility/test/node and are exactly
identical except for the package name in package.json

Depends on D83909

Hi Martin! Was wondering if you had a chance to check Daisuke's question about the UTM parameters for the compatibility tooltip's "Learn more" link?

The original inline comment is at https://phabricator.services.mozilla.com/D81474#inline-484879

Flags: needinfo?(mbalfanz)

Thanks for the ping, Micah! I guess my email filter didn't work as intended...

Anyways, I commented in phabricator. The UTM parameters look good :+1:

Flags: needinfo?(mbalfanz)
Attachment #9164273 - Attachment is obsolete: true
Attachment #9165121 - Attachment is obsolete: true
Pushed by mtigley@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/dfce9d57a546 Add tooltip in rules panel to highlight CSS compatiblity issues for particular declaration r=mtigley,daisuke,flod,ckerschb DONTBUILD https://hg.mozilla.org/integration/autoland/rev/d651a9a4ef5f Add tests for CSS Compatibility Tooltip r=mtigley,daisuke DONTBUILD
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
Regressions: 1658328
Regressions: 1660047
No longer regressions: 1660047
Regressions: 1659516

Hi,
I'm trying to test this prior to updating the MDN docs/release notes. However on FF83 and FF81 I can't get tooltip to appear. I used a few examples from the linked issues - e.g in this exmaple -webkit-font-smoothing:antialiased; should have a tooltip, but when I try it does not.

Is there something I need to do special to get the tooltips to work? If not, is there a page I can try to test this.

One further question - what level of detail does this show? Is it just "the current release version of each of these things doesn't work" or does this show all browsers and versions that don't work?

Flags: needinfo?(lelouch.cpp)

Have you toggled the flag devtools.inspector.ruleview.inline-compatibility-warning.enabled to true in about:config?
If it still doesn't appear then please file a bug with the steps to reproduce and I'll look into it asap!
Thank you

Flags: needinfo?(lelouch.cpp)

The compatibility details come from the open source repository maintained by MDN here - https://github.com/mdn/browser-compat-data
Currently the tooltip shows why a particular rule is incompatible (deprected, experimental, not implemented on all platforms) and lets you know which platforms the particular rule is supported on including the version number.

There is also a compatibility panel that can be viewed by setting the pref devtools.inspector.compatibility.enabled in about:config to true where you can see the incompatible rules and where it exists for the entire web page.
If you have any more questions, feel free to ask an I'll answer it as soon as possible.

Thank you.

Kriyszig THANKS very much. The fact this was hidden behind the preference wasn't obvious to me!

Please ignore the "request info" tag (or clear it). That was just to make sure you got this "thankyou" message.

Flags: needinfo?(lelouch.cpp)

FYI Kriyszig, FYI devtools.inspector.compatibility.enabled seems to be ignored in FF83 (at least). i.e. the compatibility pane next to layout, computed, changes etc is always on

Ah I see. Thanks for bringing it to my notice.
Thank you for using this experimental feature. If you face any problems, you can leave a comment here of file a bug yourself and we'll get on it asap.

Flags: needinfo?(lelouch.cpp)
Regressions: 1842197
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: