Closed
Bug 348856
Opened 19 years ago
Closed 19 years ago
Adding missing id's to <menupopup> elements in viewSource.xul viewPartialSource.xul viewSourceOverlay.xul
Categories
(SeaMonkey :: UI Design, enhancement)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey1.1alpha
People
(Reporter: philip.chee, Assigned: philip.chee)
References
()
Details
(Keywords: fixed-seamonkey1.1)
Attachments
(1 file)
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•19 years ago
|
||
I didn't fix the trailing whitespace nit in this patch. Should I?
![]() |
Assignee | |
Comment 2•19 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•19 years ago
|
Attachment #234010 -
Flags: superreview?(neil)
Attachment #234010 -
Flags: superreview+
Attachment #234010 -
Flags: review?(neil)
Attachment #234010 -
Flags: review+
![]() |
Assignee | |
Updated•19 years ago
|
Whiteboard: [checkin needed]
![]() |
Assignee | |
Comment 4•19 years ago
|
||
gavin says on #irc "I will notice it quicker if I get bugmail"
Comment 5•19 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
Closed: 19 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•19 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•19 years ago
|
||
Landed on MOZILLA_1_8_BRANCH.
Updated•19 years ago
|
Keywords: fixed-seamonkey1.1
You need to log in
before you can comment on or make changes to this bug.
Description
•