Closed Bug 1536676 Opened 5 years ago Closed 5 years ago

Pseudo-class locks are out of sync between Markup view context menu and Rules view

Categories

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

Product:
DevTools
Component:
Inspector: Rules
Version:
67 Branch
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: scook0+bugzilla, Assigned: rcaliman)

References

Details

(Whiteboard: [dt-q])

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:67.0) Gecko/20100101 Firefox/67.0

Steps to reproduce:

  1. Open any page (e.g. <about:blank>)
  2. Open the Devtools Inspector (Ctrl+Shift+C)
  3. In the Rules pane on the right, click "Toggle pseudo-classes" to make the pseudo-class checkboxes visible
  4. Select any page element in the Inspector (e.g. <body>)
  5. Right-click on the selected element in the Inspector, and apply any of the four pseudo-classes (e.g. "hover")
  6. Notice that yellow dots appear in the Inspector, and that corresponding checkbox in the Rules pane is now checked
  7. Right-click on the selected element again, and choose the same pseudo-class (e.g. "hover") to remove it
  8. Notice that the yellow dots disappear in the inspector, indicating that the pseudo-class is no longer applied

Actual results:

The corresponding checkbox in the Rules pane (e.g. "hover") remains checked

Expected results:

The corresponding checkbox in the Rules pane should become unchecked

Step 5 can also be replaced with checking one of the pseudo-class checkboxes in the Rules pane.

As long as the pseudo-class is unapplied via the context menu, the checkbox will fail to clear itself.

The incorrect checkbox state persists when hiding or showing the "Toggle pseudo-classes" section.

However, it re-syncs when a different page element is selected.

Component: Untriaged → Inspector
Product: Firefox → DevTools
Status: UNCONFIRMED → NEW
Component: Inspector → Inspector: Rules
Ever confirmed: true
Priority: -- → P3
Whiteboard: [dt-q]
Summary: Devtools Inspector: Removing pseudo-classes with the context menu doesn't update the corresponding checkbox → Pseudo-class locks are out of sync between Markup view context menu and Rules view
Assignee: nobody → rcaliman
Status: NEW → ASSIGNED
Depends on: 1555064

Will be fixed along with Bug 1555064. After that lands, verify and close this one.

Flags: qe-verify+

This bug was fixed along with the patch in Bug 1555064.

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Flags: qe-verify+
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.