Closed Bug 1690030 Opened 3 years ago Closed 3 years ago

Move 'View Image' on content context menu to become 'Open Image in New Tab'

Categories

(Firefox :: Menus, defect, P1)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
88 Branch
Tracking Status
firefox88 --- verified

People

(Reporter: enndeakin, Assigned: enndeakin)

References

(Blocks 1 open bug)

Details

(Whiteboard: [proton-context-menus])

Attachments

(1 file)

No description provided.
Whiteboard: [proton-context-menus]
Severity: -- → S3
Type: task → defect
OS: Unspecified → All
Priority: -- → P1
Hardware: Unspecified → Desktop

As per guidance from Vicky, for tracking, we're marking all the bugs that people are working on as P1.

Blocks: 1697470
Pushed by neil@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f7c3a7daf522
change view image on context menu to default to open in a new tab, r=Gijs,fluent-reviewers
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch

Verified that this works as expected in latest Nightly 88.0a1 across platforms (Windows 10, macOS 11.3 and Ubuntu 18.04).

Status: RESOLVED → VERIFIED

The commit message says that it should now only default to opening in a new tab. Is there still a way to open images in the same tab then? I wasn't able to glean anything from the diff in that regard, and I didn't find a corresponding pref in about.config either.

We could make it so that ctrl/command + click or middle click opens in the same tab. I thought that I had done that, but it doesn't seem that way. Regardless, that should be a new bug.

Thank you; I created a new issue at Bug 1699128. I wasn't sure how to set the milestone to the 88 Branch, but I hope the rest of the information is sufficient for tracking.

Regressions: 1699128
See Also: → 1706444
Depends on: 1706487

Major issues with the above change:

  • left click and middle click do the same now.
  • no option anymore to open link in current tab

My suggestion to approach this is to extend the whereToOpenLink() function, see related issues:

IMHO we should eventually be able to have

  • left click open in either new foreground tab or (optionally) current tab
  • middle click open either in new background tab or (optionally) new foreground tab
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: