Separator is double in HELP menu of "View Source" and "View selected Source" window

RESOLVED FIXED in Firefox 10

Status

()

Firefox
Menus
--
trivial
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Ilja Nedilko, Assigned: Vlad Tanase)

Tracking

({regression})

Trunk
Firefox 10
regression
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

6 years ago
User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0) Gecko/20100101 Firefox/4.0
Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0) Gecko/20100101 Firefox/4.0

There are two horizontal lines in between "Restart with add-ons disabled..." and "About firefox", in "View page source" (ctrl + u).

Reproducible: Always

Steps to Reproduce:
1. Go to "Firefox", "Web developer", "View page source".
2. Go to "Help menu".
3. There are two horizontal lines in between "Restart with add-ons disabled..." and "About firefox".
Actual Results:  
Two horizontal lines appear.

Expected Results:  
I assume that just one horizontal line needs to be in between those two fields.

Screenshot: http://postimage.org/image/1dzz1em4k/
(Reporter)

Comment 1

6 years ago
Sorry, made a tiny mistake in the title, it's not located in "View" menu but in "Help".

Comment 2

6 years ago
Created attachment 518452 [details]
Screenshot

Separator is double in HELP menu of "View Source" and "View selected Source" window

Updated

6 years ago
Status: UNCONFIRMED → NEW
Component: Developer Tools → Menus
Ever confirmed: true
OS: Windows 7 → All
QA Contact: developer.tools → menus
Hardware: x86_64 → All

Updated

6 years ago
Blocks: 542122
Version: unspecified → Trunk

Updated

6 years ago
Summary: Two horizontal lines in "View" menu in "View source code" → Separator is double in HELP menu of "View Source" and "View selected Source" window

Updated

6 years ago
Keywords: regression
That's weird, my build is fine, as it says Report Web Forgery inbetween the separators.

Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b13pre) Gecko/20110310 Firefox/4.0b13pre ID:20110310030427
(In reply to comment #3)
> That's weird, my build is fine, as it says Report Web Forgery inbetween the
> separators.
> 
> Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b13pre) Gecko/20110310
> Firefox/4.0b13pre ID:20110310030427

That is different then the view source screen.  Confirming the view source screen is different then the main window help menu.
(Assignee)

Comment 5

6 years ago
Created attachment 565804 [details] [diff] [review]
Patch to remove the double separator in the help menu of the view source page.

Hello,

I added a patch to correct this issue. This is my first time submitting code to firefox so I hope everything is according to the expected work flow. If there are any problems with the submitted modifications I will do my best to fix them in a timely manner.

Vlad
Attachment #565804 - Flags: review?(dolske)
(Assignee)

Comment 6

6 years ago
Created attachment 565808 [details] [diff] [review]
Patch to remove the double separator in the help menu of the view source page.

Updated the patch according to this page: https://developer.mozilla.org/en/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F

Vlad
Attachment #565804 - Attachment is obsolete: true
Attachment #565804 - Flags: review?(dolske)
Attachment #565808 - Flags: review?(dolske)
Comment on attachment 565808 [details] [diff] [review]
Patch to remove the double separator in the help menu of the view source page.

Hmm, the extra separator was added in bug 542122 (which was adding the "Restart With Add-ons Disabled…" menuitem.

The XUL looks kind of dumb at first glance, but that's because report-phishing-overlay.xul has |insertbefore="aboutSeparator"|, which addds the "Report Web Forgery…" menuitem between the two separators. The double-separator bug manifests whenever this overlay isn't included.

I suppose we could aim for a more complicated patch that preserves the current visual appearance by only sometimes removing the extra separator... But I think menu's already a bit of a mess, and the separators to put Report Web Forgery in its own block doesn't seem useful.

So let's just go with your simple patch. :)

Thanks for the patch!
Attachment #565808 - Flags: review?(dolske) → review+
Keywords: checkin-needed

Comment 8

6 years ago
Thanks for the patch! 

It has landed in mozilla-inbound and will get merged to mozilla-central in <24 hours (https://wiki.mozilla.org/Tree_Rules/Inbound#What_is_mozilla-inbound.3F), where you will then see it in the ~11th+ Oct Nightly / will be visible to end-users in Firefox 10.

https://hg.mozilla.org/integration/mozilla-inbound/rev/4d61f3ed73b9

Hopefully see you on IRC (http://irc.mozilla.org/) in #developers (my nick is edmorley) - if you'd like to fix another bug (we'd love it if you did!) but need some inspiration, pop on and we'll find something for you :-)
Assignee: nobody → vtanase.bugzilla
Status: NEW → ASSIGNED
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → Firefox 10
https://hg.mozilla.org/mozilla-central/rev/4d61f3ed73b9
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Comment 10

6 years ago
Wow this went faster than expected.

Thank you for being so responsive guys. Will definitely be dropping by on IRC and fixing some more bugs should I have the time. :-)

Comment 11

6 years ago
FWIW, this is present in the Aurora nightly and Firefox 8.
(In reply to Aleksej [:Aleksej] from comment #11)
> FWIW, this is present in the Aurora nightly and Firefox 8.

Thanks for the confirmation, though this is expected. Comment 7 mentions that this issue was introduced by bug 542122, which landed for Firefox 4b7 - so the problem has existed since then. The fix here has landed for Firefox 10 (see target milestone field at the top of this page), so won't be present in the Aurora (9) or Firefox 8 builds you tested.
You need to log in before you can comment on or make changes to this bug.