Closed Bug 1654731 Opened 3 years ago Closed 11 months ago

Preview class when selecting item in the cls autocomplete popup


(DevTools :: Inspector: Rules, enhancement)



(firefox101 fixed)

101 Branch
Tracking Status
firefox101 --- fixed


(Reporter: nchevobbe, Assigned: rfemailtesting, NeedInfo)



(Keywords: parity-chrome)


(1 file, 1 obsolete file)

It would be nice if, like in Chrome, browsing through the autocomplete results, the selected class in the popup would be previewed on screen.

Assignee: nobody → rfemailtesting

sorry I just wanted to modify my commit message, by amending the message of the existing commit, but it seems like it created a new patch 🙈😅

Attachment #9252663 - Attachment is obsolete: true
Pushed by
[devtools] Preview class when selecting item in the cls autocomplete popup r=jdescottes

Backed out for devtools failures on browser_rules_class_panel_add.js.

Push with failures:

Failure log:

Backout link:

[task 2022-04-23T11:49:26.499Z] 11:49:26     INFO - TEST-PASS | devtools/client/inspector/rules/test/browser_rules_class_panel_add.js | Success: untitled - 
[task 2022-04-23T11:49:26.499Z] 11:49:26     INFO - Check the state of the DOM node
[task 2022-04-23T11:49:26.500Z] 11:49:26     INFO - Buffered messages finished
[task 2022-04-23T11:49:26.501Z] 11:49:26     INFO - TEST-UNEXPECTED-FAIL | devtools/client/inspector/rules/test/browser_rules_class_panel_add.js | The DOM node has the right className - Got "class a clas class2 class3", expected "class a class2 class3"
[task 2022-04-23T11:49:26.502Z] 11:49:26     INFO - Stack trace:
[task 2022-04-23T11:49:26.502Z] 11:49:26     INFO - chrome://mochikit/content/browser-test.js:test_is:1422
[task 2022-04-23T11:49:26.503Z] 11:49:26     INFO - chrome://mochitests/content/browser/devtools/client/inspector/rules/test/browser_rules_class_panel_add.js:null:102
[task 2022-04-23T11:49:26.503Z] 11:49:26     INFO - chrome://mochikit/content/browser-test.js:handleTask:989
[task 2022-04-23T11:49:26.503Z] 11:49:26     INFO - chrome://mochikit/content/browser-test.js:_runTaskBasedTest:1061
[task 2022-04-23T11:49:26.504Z] 11:49:26     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1189
[task 2022-04-23T11:49:26.504Z] 11:49:26     INFO - chrome://mochikit/content/browser-test.js:nextTest/<:978
[task 2022-04-23T11:49:26.505Z] 11:49:26     INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/<:1041
[task 2022-04-23T11:49:26.506Z] 11:49:26     INFO - Check the content of the class panel
[task 2022-04-23T11:49:26.506Z] 11:49:26     INFO - Not taking screenshot here: see the one that was previously logged
[task 2022-04-23T11:49:26.508Z] 11:49:26     INFO - TEST-UNEXPECTED-FAIL | devtools/client/inspector/rules/test/browser_rules_class_panel_add.js | The panel contains the expected number of checkboxes - Got 5, expected 4
[task 2022-04-23T11:49:26.508Z] 11:49:26     INFO - Stack trace:
[task 2022-04-23T11:49:26.509Z] 11:49:26     INFO - chrome://mochikit/content/browser-test.js:test_is:1422
[task 2022-04-23T11:49:26.509Z] 11:49:26     INFO - chrome://mochitests/content/browser/devtools/client/inspector/rules/test/head.js:checkClassPanelContent:647
[task 2022-04-23T11:49:26.510Z] 11:49:26     INFO - chrome://mochitests/content/browser/devtools/client/inspector/rules/test/browser_rules_class_panel_add.js:null:105
[task 2022-04-23T11:49:26.510Z] 11:49:26     INFO - chrome://mochikit/content/browser-test.js:handleTask:989
[task 2022-04-23T11:49:26.511Z] 11:49:26     INFO - chrome://mochikit/content/browser-test.js:_runTaskBasedTest:1061
[task 2022-04-23T11:49:26.511Z] 11:49:26     INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1189
[task 2022-04-23T11:49:26.511Z] 11:49:26     INFO - chrome://mochikit/content/browser-test.js:nextTest/<:978
[task 2022-04-23T11:49:26.512Z] 11:49:26     INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/<:1041
[task 2022-04-23T11:49:26.513Z] 11:49:26     INFO - TEST-PASS | devtools/client/inspector/rules/test/browser_rules_class_panel_add.js | Checkbox 0 has the right class name -
Flags: needinfo?(rfemailtesting)
Pushed by
[devtools] Preview class when selecting item in the cls autocomplete popup r=jdescottes
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → 101 Branch
You need to log in before you can comment on or make changes to this bug.