Closed Bug 1499755 Opened 2 years ago Closed 2 years ago
Clicking on inline stylesheet links in then rule-view does not open the right stylesheet in the style-editor
46 bytes, text/x-phabricator-request
|Details | Review|
STR: - open http://m8y.org/tmp/testcase466.xhtml - right-click on one of the paragraphs and select "inspect element" - in the rule-view, next to the CSS rule that applies to the p elements, click on the inline:2 link --> expected: the style-editor opens up and shows the right style-sheet, with the cursor at the right location --> actual: the style-editor opens up, but does not show the right style-sheet.
Assignee: nobody → pbrosset
Status: NEW → ASSIGNED
The code we had to jump to the style-editor when a rule-view source link is clicked did not make any distinction between multiple inline stylesheets. If you had many of them, they would all have the same url (i.e. the url of the document, because they are inline). And we were matching stylesheets in the style-editor by url, so we would always select the first inline stylesheet. This change makes use of the fact that the style-editor's selectStyleSheet function also accept a StyleSheetFront object. When passing this object, there can be no confusion, because they're all different. Now, I'm only doing this for inlin stylesheets because other stylesheets have unique urls and it's important to preserve the previous logic since source-maps may also be involved. I'm taking this opportunity to re-enable browser_rules_style-editor-link.js which had been disabled a long time ago, and removing a part that just doesn't work anymore at all apparently.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/3a70a65a75cf Select the right inline stylesheet on rule-view source url click; r=gl
You need to log in before you can comment on or make changes to this bug.