Closed Bug 417476 Opened 12 years ago Closed 12 years ago

This Frame context submenu needs menu icons

Categories

(Firefox :: Theme, defect, trivial)

x86
Linux
defect
Not set
trivial

Tracking

()

VERIFIED FIXED
Firefox 3 beta4

People

(Reporter: adelfino, Assigned: reed)

References

Details

Attachments

(2 files)

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
Severity: normal → trivial
Component: Menus → Theme
Version: unspecified → Trunk
QA Contact: menus → theme
Blocks: 405165
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attached patch patch - v1Splinter Review
Assignee: nobody → reed
Status: NEW → ASSIGNED
Attachment #305735 - Flags: review?(rflint)
Attachment #305735 - Flags: review?(rflint) → review+
Attachment #305735 - Flags: approval1.9?
Comment on attachment 305735 [details] [diff] [review]
patch - v1

a1.9+=damons
Attachment #305735 - Flags: approval1.9? → approval1.9+
Keywords: checkin-needed
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: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3 beta4
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.