Adding missing id's to <menupopup> elements in viewSource.xul viewPartialSource.xul viewSourceOverlay.xul
RESOLVED
FIXED
in seamonkey1.1alpha
Status
People
(Reporter: philip.chee, Assigned: philip.chee)
Tracking
({fixed-seamonkey1.1})
Firefox Tracking Flags
(Not tracked)
Details
(URL)
Attachments
(1 attachment)
|
2.04 KB,
patch
|
neil
:
review+
neil
:
superreview+
kairo
:
approval-seamonkey1.1+
|
Details | Diff | Splinter Review |
I need these IDs as the JSView extension (ported from Firefox to SeaMonkey) needs these to successfully overlay the view source windows. Patch coming up.
|
(Assignee) | |
Comment 1•13 years ago
|
||
Created attachment 234010 [details] [diff] [review] Patch v1.0 I didn't fix the trailing whitespace nit in this patch. Should I?
|
|
(Assignee) | |
Comment 2•13 years ago
|
||
Comment on attachment 234010 [details] [diff] [review] Patch v1.0 obvious one-liners, asking for r/sr
Attachment #234010 -
Flags: superreview?(neil)
Attachment #234010 -
Flags: review?(neil)
|
||
Updated•13 years ago
|
Attachment #234010 -
Flags: superreview?(neil)
Attachment #234010 -
Flags: superreview+
Attachment #234010 -
Flags: review?(neil)
Attachment #234010 -
Flags: review+
|
|
(Assignee) | |
Updated•13 years ago
|
Whiteboard: [checkin needed]
|
|
(Assignee) | |
Comment 4•13 years ago
|
||
gavin says on #irc "I will notice it quicker if I get bugmail"
|
||
Comment 5•13 years ago
|
||
mozilla/xpfe/browser/resources/content/viewSource.xul 1.57 mozilla/xpfe/browser/resources/content/viewPartialSource.xul 1.8 mozilla/xpfe/browser/resources/content/viewSourceOverlay.xul 1.28
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed]
Target Milestone: --- → seamonkey1.1alpha
Comment on attachment 234010 [details] [diff] [review] Patch v1.0 first-a = me for seamonkey 1.1 final, need one more
Attachment #234010 -
Flags: approval-seamonkey1.1?
|
|
||
Comment 7•13 years ago
|
||
Comment on attachment 234010 [details] [diff] [review] Patch v1.0 a=me for SeaMonkey 1.1
Attachment #234010 -
Flags: approval-seamonkey1.1? → approval-seamonkey1.1+
|
||
Comment 8•12 years ago
|
||
Landed on MOZILLA_1_8_BRANCH.
|
||
Updated•12 years ago
|
Keywords: fixed-seamonkey1.1
You need to log in
before you can comment on or make changes to this bug.
Description
•