Closed Bug 375843 Opened 16 years ago Closed 16 years ago

Report Web Forgery menu item changed location

Categories

(Firefox :: Menus, defect)

Product:
Firefox
Component:
Menus
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 3 alpha4

People

(Reporter: polidobj, Assigned: philor)

References

Details

(Keywords: regression)

Attachments

(1 file)

Fix v.1
2.95 KB, patch
asaf
: review+
Details | Diff | Splinter Review 
The Report Web Forgery menu item has moved from being below Report Broken Web Site to being below Check for Updates.  This is because the separators were switched. 

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a4pre) Gecko/20070328 Minefield/3.0a4pre ID:2007032812 [cairo]
Attached patch Fix v.1 — — Splinter Review 
I think I'll just assume that the reason why we have an aboutSeparator, when every other app has a menu_HelpAboutSeparator, is better than the reason why we and Tb have/had a .*aboutseparator above the thing above the separator above About.
Assignee: nobody → philringnalda
Status: NEW → ASSIGNED
Attachment #260094 - Flags: review?(mano)
OS: Windows XP → All
Target Milestone: --- → Firefox 3 alpha4
Comment on attachment 260094 [details] [diff] [review]
Fix v.1

r=mano.
Attachment #260094 - Flags: review?(mano) → review+
browser/base/content/baseMenuOverlay.xul: 1.16
browser/components/safebrowsing/content/report-phishing-overlay.xul;/cvsroot/mozilla/browser/components/safebrowsing/content/report-phishing-overlay.xul: 1.7
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.