Closed
Bug 994989
Opened 10 years ago
Closed 10 years ago
Move Save as pdf to the Page menu
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
Firefox 31
People
(Reporter: wesj, Assigned: mcomella)
Details
Attachments
(1 file, 1 obsolete file)
5.26 KB,
patch
|
wesj
:
review+
|
Details | Diff | Splinter Review |
Save as pdf is in the tools menu for historical reasons. It really makes a whole lot more sense in the page menu.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → michael.l.comella
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8406293 -
Flags: review?(wjohnston)
Reporter | ||
Comment 2•10 years ago
|
||
Comment on attachment 8406293 [details] [diff] [review] Move Save as PDF to Page menu. Review of attachment 8406293 [details] [diff] [review]: ----------------------------------------------------------------- There's two ther menus that need updated: http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/resources/menu-large-v11/browser_app_menu.xml#78 http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/resources/menu-xlarge-v11/browser_app_menu.xml#79 (Yes this is annoying...) I don't see any tests for this...
Attachment #8406293 -
Flags: review?(wjohnston) → review-
Assignee | ||
Comment 3•10 years ago
|
||
Sorry, that was sloppy on my part. Questions: * Should I remove the drawable file? * Should I be removing "android:icon" (as I'm doing)? It seems to be ignored, and, if we don't remove the drawable, it'd be harder to forget the drawable exists (though, I guess it might require more runtime resources).
Attachment #8406313 -
Flags: review?(wjohnston)
Assignee | ||
Updated•10 years ago
|
Attachment #8406293 -
Attachment is obsolete: true
Assignee | ||
Comment 4•10 years ago
|
||
(In reply to Wesley Johnston (:wesj) from comment #2) > (Yes this is annoying...) I don't see any tests for this... Created bug 996227.
Reporter | ||
Updated•10 years ago
|
Attachment #8406313 -
Flags: review?(wjohnston) → review+
Assignee | ||
Comment 5•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/c80c6685d1e0
Comment 6•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/c80c6685d1e0
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 31
Verified as fixed on Nightly 31.0a1(2014-04-16).
Status: RESOLVED → VERIFIED
Comment 8•10 years ago
|
||
added to https://docs.google.com/a/mozilla.com/spreadsheet/ccc?key=0Amy0OSD97CR6dGF4ZkhVaFFnaEI1cWhpR0dyM1hlaUE&usp=drive_web#gid=3 for joni savage to document in SUMO
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•