1849881 - Copy Location should support css sourcemaps

Status: RESOLVED FIXED

(DevTools :: Inspector, defect, P3)

Description

[tajinder.gill]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:109.0) Gecko/20100101 Firefox/116.0

Steps to reproduce:

In the rules part of the Inspector on a site with CSS sourcemaps enabled and "Show original sources" enabled. Hover over the source link and choose "Copy Location".

Actual results:

The location of the rendered css file is copied.

Expected results:

If "Show Original Sources" is enabled this arguable should copy what is displayed which is the source file path. This isn't the case however the rendered file path is always copied. If this isn't desirable then "Copy Source Location" field should be added as this is a common use case for developers.

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

The context menu is handled in https://searchfox.org/mozilla-central/rev/479f4762b8c537892269dbf4442dd33cc01a525c/devtools/client/styleeditor/StyleEditorUI.sys.mjs#807-811

#copyUrl = () => {
  if (this.#contextMenuStyleSheet) {
    lazy.copyString(this.#contextMenuStyleSheet.href);
  }
};

Comment 2

[Ardelean Oana, Desktop Test Engineering [:oardelean]]

Thank you for the enhancement suggestion.
Setting as NEW so the developing team can have a look.

Comment 3

[Julian Descottes [:jdescottes]]

STRs:

• go to MDN
• open style editor
• make sure "Show original sources" is checked
• open inspector
• right click on any css link pointing to _themes.css
• copy source location

Expected result: should copy the linl to _themes.css
Actual result: copies a link to the generated file

Note that in the style editor right clicking on the file and using Copy URL copies the right URL.

Comment 4

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1849881 - [devtools] Make Copy Location context menu handle original file. r=#devtools-reviewers.

Copy whatever URL is being displayed to the user.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/773b8562a6c2
[devtools] Make Copy Location context menu handle original file. r=devtools-reviewers,ochameau.

Comment 6

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/773b8562a6c2