Closed
Bug 714342
Opened 12 years ago
Closed 12 years ago
page-proxy-stack should be a <hbox> rather than a <stack>
Categories
(Firefox :: Address Bar, defect)
Firefox
Address Bar
Tracking
()
RESOLVED
FIXED
Firefox 12
People
(Reporter: dao, Assigned: murali.sr92)
Details
(Whiteboard: [good first bug][mentor=dao][lang=xul])
Attachments
(1 file, 2 obsolete files)
1.60 KB,
patch
|
dao
:
review+
|
Details | Diff | Splinter Review |
http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser.xul#539 page-proxy-stack contains only one element (page-proxy-favicon), there's nothing to be stacked. We should use a plain box element for this, not a stack element. We should however keep the "page-proxy-stack" id for backwards-compatibility with add-ons.
Hi, I would like to work on this bug. Can you please mentor me? This is my first bug. Thanks a lot!
Reporter | ||
Comment 2•12 years ago
|
||
Hi! Do you already have a copy of the source code? https://developer.mozilla.org/en/Mozilla_Source_Code_%28Mercurial%29 has some information on this.
I already got the source code from https://developer.mozilla.org/En/Simple_Firefox_build . Could you please tell me whether it is the same? Or should I follow the instructions from the link you provided? Thanks!
Reporter | ||
Comment 4•12 years ago
|
||
It's the same. You can now open browser/base/content/browser.xul in your editor, find the line containing '<stack id="page-proxy-stack"' and make this a "hbox" element instead of a "stack" element.
Reporter | ||
Comment 6•12 years ago
|
||
Comment on attachment 590664 [details] [diff] [review] page-proxy-stack made into a <hbox> element >+ <hbox id="page-proxy-stack" > onclick="PageProxyClickHandler(event);"> nit-pick: please adjust the second line's indentation so that the attributes line up again
Reporter | ||
Updated•12 years ago
|
Assignee: nobody → murali.sr92
Alignment fixed...?
Attachment #590664 -
Attachment is obsolete: true
Reporter | ||
Comment 8•12 years ago
|
||
Comment on attachment 590665 [details] [diff] [review] page-proxy-stack made into a <hbox> element (alignment fixed) You seem to have modified the patch. Instead, you need to modify the source and generate a new patch.
So sorry, I did not know it made a difference.
Attachment #590665 -
Attachment is obsolete: true
Reporter | ||
Comment 10•12 years ago
|
||
Comment on attachment 590666 [details] [diff] [review] page-proxy-stack made into a <hbox> element Thanks!
Attachment #590666 -
Flags: review+
Assignee | ||
Comment 11•12 years ago
|
||
Thank you so much for guiding me!! :)
Reporter | ||
Comment 12•12 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/79cb2db62a9b
Target Milestone: --- → Firefox 12
Comment 13•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/79cb2db62a9b Thank you for the patch! This will be in tomorrow's nightly. If you need any inspiration for things to fix next, pop on #developers @ irc.mozilla.org :-)
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•