Closed Bug 255256 Opened 21 years ago Closed 13 years ago

disallow context menus script setting thing by default

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: asa, Unassigned)

Details

Attachments

(1 file)

We should set the pref by default so sites can't hijack the context menu. If whoever is in there doing that would like to make me happy, they could also make that window resizeable.
(and maybe we should also just disable raising and lowering of windows and moving or resizing existing windows too. all of this should help to prevent spoofing)
Some more annoyances that should be disabled by default: Bug 186708 resizing windows that have toolbars Bug 177838 creating non-resizable windows
I agree. That option should be set by default.
Attached patch PatchSplinter Review
Asa, do you want to drive this in?
Assignee: firefox → gavin.sharp
Status: NEW → ASSIGNED
Attachment #180425 - Flags: review?(mconnor)
Whiteboard: [patch-r?]
Comment on attachment 180425 [details] [diff] [review] Patch This'll have unexpected behaviour with the new prefwindow code. Disable common annoyances would then do nothing with this pref, then set it back. Not acceptable behaviour. Jesse, you had something written up about disabling some of these by default and removing that poorly defined checkbox, care to share?
Attachment #180425 - Flags: review?(mconnor) → review-
Attachment #180425 - Attachment is obsolete: true
OS: Windows XP → All
Hardware: PC → All
Whiteboard: [patch-r?]
Version: unspecified → Trunk
(In reply to comment #5) > This'll have unexpected behaviour with the new prefwindow code. Disable common > annoyances would then do nothing with this pref, then set it back. Not > acceptable behaviour. I think it should be noted that the new pref window already does this for some other prefs (bug 284079). That doesn't make this the right fix, of course, but it emphasizes the need for a better general solution.
Comment on attachment 180425 [details] [diff] [review] Patch This should work now that the prefs UI was redone.
Attachment #180425 - Attachment is obsolete: false
Attachment #180425 - Flags: review- → review?
Attachment #180425 - Flags: review? → review?(mconnor)
Assignee: gavin.sharp → nobody
Status: ASSIGNED → NEW
So the patch can now be reviewed again?
Many sites now depend on this behavior, for example Google Docs provides their own context menu that is specific to the workflow/actions of the application. Changing this will break those sites.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: