Closed Bug 1845037 Opened 1 year ago Closed 3 months ago

DevTools screenshot and Firefox screenshots are saved in different locations

Categories

(DevTools :: General, task)

task

Tracking

(firefox127 fixed)

RESOLVED FIXED
127 Branch
Tracking Status
firefox127 --- fixed

People

(Reporter: jdescottes, Assigned: jdescottes)

References

(Depends on 1 open bug)

Details

Attachments

(1 file)

See https://discourse.mozilla.org/t/screenshot-node-saves-screenshot-to-pictures-instead-of-downloads-folder/121286/2

After Bug 1072345, DevTools will attempt to save screenshots (eg node screenshots) in the "screenshots" directory provided by the platform (Services.dirsvc.get("Scrnshts", Ci.nsIFile)).

In practice, for macos it means we save in Pictures instead of Downloads previously.
However the main Firefox screenshot feature (right click on page > take screenshot) saves screenshots under Downloads for the same scenario.

Should we align the two? And if we think the Downloads location makes more sense, I'm not against reverting the DevTools change, but I just think we should have a consistent behavior here.

Sam, as owner of the Screenshots component, do you have an opinion on this question?

Flags: needinfo?(sfoster)

(we have another bug for this in which Sam provided feedback, I'm trying to find it)

(In reply to Nicolas Chevobbe [:nchevobbe] from comment #2)

(we have another bug for this in which Sam provided feedback, I'm trying to find it)

This was bug 1838053

Flags: needinfo?(sfoster)

Ah thanks, I couldn't find any duplicate. So I guess we can block on Bug 1839949, and once that's implemented, use this value as the screenshot location.

Depends on: 1839949
No longer depends on: 1839949
Depends on: 1838053

Moving back to triage.

Changing the default folder to the OS Image folder was intentional because we are not "downloading" anything in terms of network activity, but the whole UX makes the user feel like there is a download happening. Mostly it shows the file in the download popup of the browser.

I think the rationale to keep the Downloads folder for the Firefox screenshot feature also applies here:

we actually call the browser.downloads extension API to "download" the file, which triggers the download animation and highlight on the download toolbar button.

Based on this, users will continue to expect the images to show up in the Download folder, and not in the Pictures folder.
We should consider reverting this.

Whiteboard: [devtools-triage]

As mentioned by :zcorpan on Slack, macos' built in screenshot feature saves to Desktop by default. I still feel like Downloads makes more sense based on the rationale above, and ultimately users should be able to set a custom folder via Bug 1838053.

In triage, we agreed to revert the change and save them to the downloads folder to be consistent with Firefox screenshot.

Whiteboard: [devtools-triage]

Revert the DevTools bits from Bug 1072345 to be consistent with the Screenshots feature from Firefox.

Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Pushed by jdescottes@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9991fc63747c
[devtools] Revert DevTools screenshots to use the default downloads directory r=devtools-reviewers,ochameau
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 127 Branch
You need to log in before you can comment on or make changes to this bug.