Closed
Bug 223295
Opened 21 years ago
Closed 21 years ago
<dialog>s don't need trailing <separator>s
Categories
(SeaMonkey :: MailNews: Message Display, defect)
SeaMonkey
MailNews: Message Display
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.7beta
People
(Reporter: neil, Assigned: Stefan.Borggraefe)
Details
Attachments
(1 file)
|
2.45 KB,
patch
|
neil
:
review+
mscott
:
superreview+
|
Details | Diff | Splinter Review |
As per bug 187466 if dialog buttons are too close to dialog content it's a theme
issue. However there are still old dialogs out there with trailing separators,
I see AccountManager.xul, SmtpServerList.xul, subscribe.xul and
FilterEditor.xul; there may be others.
|
Assignee | |
Comment 2•21 years ago
|
||
I searched with
find . -name "*.xul" | xargs grep -B 3 \</dialog\>
and then investigated the output. This patch removes all trailing separators I
found this way.
|
|
Assignee | |
Updated•21 years ago
|
Attachment #134309 -
Flags: superreview?(mscott)
Attachment #134309 -
Flags: review?(neil.parkwaycc.co.uk)
|
|
Assignee | |
Comment 3•21 years ago
|
||
Comment on attachment 134309 [details] [diff] [review]
Fix
AccountManager.xul is not included in this patch, because it will be fixed by
the checkin for bug 222940.
|
Reporter | |
Updated•21 years ago
|
Attachment #134309 -
Flags: review?(neil.parkwaycc.co.uk) → review+
|
||
Updated•21 years ago
|
Attachment #134309 -
Flags: superreview?(mscott) → superreview+
|
|
Assignee | |
Comment 4•21 years ago
|
||
Scott, this bug needs someone with CVS write access to TB. Could you check this
in? Thanks!
OS: Windows 2000 → All
Hardware: PC → All
|
|
||
Comment 5•21 years ago
|
||
I just checked this in
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.7beta
|
||
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•