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
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.
Version: 7 → Trunk
(In reply to rsx11m from comment #2) > Apparently fall-out from bug 654222. Nope, bug 680695.
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?
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 11.0
Attachment #574546 - Flags: approval-comm-aurora? → approval-comm-aurora+
Checked into aurora: http://hg.mozilla.org/releases/comm-aurora/rev/ad8fbea7f9e2
You need to log in before you can comment on or make changes to this bug.