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

RESOLVED FIXED in Thunderbird 11.0

Status

Thunderbird
Message Reader UI
--
major
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: Thomas D. (currently busy elsewhere; needinfo?me), Assigned: squib)

Tracking

({regression})

Trunk
Thunderbird 11.0
regression
Bug Flags:
in-testsuite +

Thunderbird Tracking Flags

(thunderbird10 fixed)

Details

Attachments

(1 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

Comment 2

6 years ago
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
(Assignee)

Comment 3

6 years ago
(In reply to rsx11m from comment #2)
> Apparently fall-out from bug 654222.

Nope, bug 680695.
Blocks: 680695
No longer blocks: 654222
(Assignee)

Comment 4

6 years ago
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
(Assignee)

Comment 5

6 years ago
Created attachment 574546 [details] [diff] [review]
Fix this and test it

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+
(Assignee)

Updated

6 years ago
Attachment #574546 - Flags: approval-comm-aurora?
(Assignee)

Comment 7

6 years ago
Checked in: http://hg.mozilla.org/comm-central/rev/c746aa0555d8
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite+
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
status-thunderbird10: --- → fixed
You need to log in before you can comment on or make changes to this bug.