Closed
Bug 496884
Opened 16 years ago
Closed 15 years ago
Collapsed summary view header hides message summary for threads with long subjects
Categories
(Thunderbird :: Message Reader UI, defect)
Thunderbird
Message Reader UI
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0b4
People
(Reporter: standard8, Assigned: davida)
References
Details
(Whiteboard: [no l10n impact])
Attachments
(2 files, 1 obsolete file)
52.67 KB,
image/png
|
Details | |
1.42 KB,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
STR:
1) Get a message thread which has a very long subject line.
2) View collapsed thread on that message thread.
3) Find you can't read the summary of the message.
4) Make Thunderbird narrower, find it is even worse.
Best explained by a picture. Note the lack of scrollbars or anything.
Flags: blocking-thunderbird3+
Comment 1•16 years ago
|
||
Since this is not a regression, I don't think we'd hold b3 for this. Retargetting to b4.
Target Milestone: Thunderbird 3.0b3 → Thunderbird 3.0b4
Updated•15 years ago
|
Whiteboard: [no l10n impact]
Assignee | ||
Comment 2•15 years ago
|
||
You complain, you get to review the fix.
Attachment #398498 -
Flags: review?(bugzilla)
Assignee | ||
Updated•15 years ago
|
Whiteboard: [no l10n impact] → [no l10n impact][has patch, needs review standard8]
Reporter | ||
Comment 3•15 years ago
|
||
Comment on attachment 398498 [details] [diff] [review]
the fix
The contentDocument/defaultView doesn't get destroyed and recreated each time we show something different in the summary pane.
Hence this adds more listeners every time we show the summary view.
Attachment #398498 -
Flags: review?(bugzilla) → review-
Reporter | ||
Comment 4•15 years ago
|
||
Oh, and the line within the function should be two-space indented.
Reporter | ||
Updated•15 years ago
|
Whiteboard: [no l10n impact][has patch, needs review standard8] → [no l10n impact][needs new patch]
Assignee | ||
Comment 5•15 years ago
|
||
nice catch. how's this? (in general, i don't like inline JS in HTML, but creating a whole new file & make patch just for this seemed overkill)
Attachment #398498 -
Attachment is obsolete: true
Attachment #398683 -
Flags: review?(bugzilla)
Assignee | ||
Updated•15 years ago
|
Whiteboard: [no l10n impact][needs new patch] → [no l10n impact][has patch, needs review standard8]
Reporter | ||
Updated•15 years ago
|
Attachment #398683 -
Flags: review?(bugzilla) → review+
Reporter | ||
Comment 6•15 years ago
|
||
Comment on attachment 398683 [details] [diff] [review]
patch v2
That's better. r=Standard8.
I've checked this in as well:
http://hg.mozilla.org/comm-central/rev/478bce596df0
Reporter | ||
Updated•15 years ago
|
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Whiteboard: [no l10n impact][has patch, needs review standard8] → [no l10n impact]
You need to log in
before you can comment on or make changes to this bug.
Description
•