|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
59 bytes, text/x-review-board-request
|Details | Review|
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.
Assignee: nobody → jaws
Status: NEW → ASSIGNED
status-firefox57: affected → wontfix
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.
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+
You need to log in before you can comment on or make changes to this bug.