Closed Bug 698272 Opened 9 years ago Closed 9 years ago

Long attachment file name screws up message reader UI for messages with single attachment

Categories

(Thunderbird :: Message Reader UI, defect, major)

defect
Not set
major

Tracking

(thunderbird10 fixed)

RESOLVED FIXED
Thunderbird 11.0
Tracking Status
thunderbird10 --- fixed

People

(Reporter: bugzilla2007, Assigned: squib)

References

Details

(Keywords: regression)

Attachments

(1 file)

Regression (works in TB 7, broken on trunk, didn't check other versions):
For messages with a single attachment, attachment file names that are longer than the dedicated space on the attachment pane header will screw up the whole message reader UI by pushing message header and message body to the right beyond the screen.

STR

1) from main 3-pane, view message with single attachment which has a long file name
2) drag the border of the folder pane to the right to reduce the width of message list and message preview

Actual result
- we no longer truncate the middle of the clickable attachment file name on attachment header bar (regression, works in TB 7, broken on trunk, didn't check other versions)
- message header and message body are pushed to the right beyond the screen -> whole UI is screwed up

Expected result
- truncate the middle of long attachment file name on attachment header bar (as we do in TB 7)
- thus avoid all the screwed-UI problems
For smaller screens, this will make the message reader UI completely unusable without any way of recovery -> major!
Severity: normal → major
Summary: Long attachment file name screw up message reader UI for messages with single attachment → Long attachment file name screws up message reader UI for messages with single attachment
Apparently fall-out from bug 654222. I didn't see this with multiple attachments for testing bug 697870, but then, didn't test it with a single attachment either.
Blocks: 654222
Version: 7 → Trunk
(In reply to rsx11m from comment #2)
> Apparently fall-out from bug 654222.

Nope, bug 680695.
Blocks: 680695
No longer blocks: 654222
Taking. This is really easy to fix, but I'm going to write a test for it as penance for screwing it up to begin with.
Assignee: nobody → squibblyflabbetydoo
Here's a patch, complete with tests that fail without the patch and pass with it. Thankfully, it's a very simple patch too!
Attachment #574546 - Flags: review?(bwinton)
Comment on attachment 574546 [details] [diff] [review]
Fix this and test it

Review of attachment 574546 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, fixes the bug, and has a test.  r=me!

Thanks,
Blake.
Attachment #574546 - Flags: review?(bwinton) → review+
Attachment #574546 - Flags: approval-comm-aurora?
Checked in: http://hg.mozilla.org/comm-central/rev/c746aa0555d8
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 11.0
Attachment #574546 - Flags: approval-comm-aurora? → approval-comm-aurora+
You need to log in before you can comment on or make changes to this bug.