In the Inspector, click (deselect) Only Show User Styles in the Style Inspector (computed view). Expand a property to reveal a rule with a link to html.css. Click the html.css link. Expected: Should see html.css with the relevant rule displayed either in the source editor or view source. Actual: Source Editor opens with an empty source editor.
Test case: 1. Inspect the Google logo 2. Open computed view 3. Expand "display" 4. Click on html.css The style editor will open with a list of inline stylesheets. Because html.css is not listed in the style editor we do not open it. Seems like we need to be able to switch the style editor into system mode so that system stylesheets can be loaded.
Created attachment 604981 [details] [diff] [review] Patch We decided to open these links in view source because we don't want users to be editing UA css.
Comment on attachment 604981 [details] [diff] [review] Patch Missing tests
Created attachment 606294 [details] [diff] [review] Tests now included
Comment on attachment 606294 [details] [diff] [review] Tests now included Review of attachment 606294 [details] [diff] [review]: ----------------------------------------------------------------- Patch looks good and all tests pass. One general comment: tests use arguments.callee which is deprecated. ::: browser/devtools/highlighter/inspector.jsm @@ +1031,2 @@ > > + // Array.prototype.indexOf always returns -1 here so we loop through Is this comment needed? ::: browser/devtools/styleinspector/test/browser_ruleview_bug_703643_context_menu_copy.js @@ -46,5 @@ > > // Make sure the inspector is open. > ok(InspectorUI.inspecting, "Inspector is highlighting"); > ok(!InspectorUI.treePanel.isOpen(), "Inspector Tree Panel is not open"); > - ok(!InspectorUI.isSidebarOpen, "Inspector Sidebar is not open"); Why is this here?
Created attachment 607231 [details] [diff] [review] Fixed orange
Mike: the latest patch you submitted here is empty.
Created attachment 609304 [details] [diff] [review] patch included Patch now included.