Open
Bug 1396992
Opened 7 years ago
Updated 2 years ago
Save Page As... should be disabled for preferences, new tab, and add-ons (all about: pages ?)
Categories
(Firefox :: Menus, enhancement, P3)
Firefox
Menus
Tracking
()
NEW
Tracking | Status | |
---|---|---|
firefox57 | --- | wontfix |
People
(Reporter: asa, Unassigned)
Details
Attachments
(1 file)
The main menu lets you save these about pages but that's not a useful feature. The menu item should be disabled for about: pages or at least the main ones.
Updated•7 years ago
|
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
The patch I'm attaching fixes this (not for about:newtab because we don't show that page as a secure-internal page). It shims this in through where we update the page identity UI which might not be the best place but it limits it to the only place where we make that distinction right now.
Updated•7 years ago
|
Priority: -- → P1
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8924301 [details] Bug 1396992 - Disable the save page command when on secure internal pages. https://reviewboard.mozilla.org/r/195546/#review200776 The implementation lgtm. Is there a context for this change though? If Save's not useful on these pages, then Email Link, Print, Text Encoding, and Page Info aren't either I would think... I'm not really suggesting that we disable them too, but this bug seems kind of arbitrary to me, separate from a context of how the browser UI should change on these pages.
Attachment #8924301 -
Flags: review?(adw) → review+
Updated•5 years ago
|
Assignee: jaws → nobody
Status: ASSIGNED → NEW
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•