Closed
Bug 417476
Opened 17 years ago
Closed 17 years ago
This Frame context submenu needs menu icons
Categories
(Firefox :: Theme, defect)
Tracking
()
VERIFIED
FIXED
Firefox 3 beta4
People
(Reporter: adelfino, Assigned: reed)
References
Details
Attachments
(2 files)
|
(deleted),
image/png
|
Details | |
|
(deleted),
patch
|
rflint
:
review+
damons
:
approval1.9+
|
Details | Diff | Splinter Review |
Screenshot showing no icons in the menu.
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b4pre) Gecko/2008021405 Minefield/3.0b4pre
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9b4pre) Gecko/2008021405 Minefield/3.0b4pre
This frame context submenu needs menu icons.
Firefox main window menus has icons that can (and should) be used in This Frame context submenu:
Open Frame in New Window
Open Frame in New Tab
Reload Frame
Bookmark This Frame
Save Frame As...
Print Frame...
View Frame Info
Reproducible: Always
|
Reporter | |
Updated•17 years ago
|
Severity: normal → trivial
Component: Menus → Theme
Version: unspecified → Trunk
|
|
Reporter | |
Comment 1•17 years ago
|
||
|
||
Updated•17 years ago
|
QA Contact: menus → theme
|
Assignee | |
Updated•17 years ago
|
|
|
Assignee | |
Comment 2•17 years ago
|
||
|
||
Updated•17 years ago
|
Attachment #305735 -
Flags: review?(rflint) → review+
|
||
Comment 3•17 years ago
|
||
Comment on attachment 305735 [details] [diff] [review]
patch - v1
mpa=mano.
|
|
Assignee | |
Updated•17 years ago
|
Attachment #305735 -
Flags: approval1.9?
|
||
Comment 4•17 years ago
|
||
Comment on attachment 305735 [details] [diff] [review]
patch - v1
a1.9+=damons
Attachment #305735 -
Flags: approval1.9? → approval1.9+
|
|
Assignee | |
Updated•17 years ago
|
Keywords: checkin-needed
|
|
Assignee | |
Comment 5•17 years ago
|
||
Checking in browser/base/content/browser-context.inc;
/cvsroot/mozilla/browser/base/content/browser-context.inc,v <-- browser-context.inc
new revision: 1.40; previous revision: 1.39
done
Checking in browser/themes/gnomestripe/browser/browser.css;
/cvsroot/mozilla/browser/themes/gnomestripe/browser/browser.css,v <-- browser.css
new revision: 1.194; previous revision: 1.193
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3 beta4
|
|
Reporter | |
Updated•17 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•