In bug 450302 this change was made to the label containing &filterHeader.label; : - <label control="filterTree">&filterHeader.label;</label> + <hbox> + <label id="filterListLabel" control="filterList" value="&filterHeader.label;" crop="end" flex="1"></label> + <label id="countBox"/> + </hbox> Thus, it no longer wraps if the dialog is too small. That is also exaggerated by the fact that the countBox label occupies the same grid cell and may force the filterHeader.label to be cropped if there are many filters (i.e. the countbox label has a long string).
This may have a bigger impact on languages that have that string longer than the en-US locale.
Severity: minor → normal
Status: NEW → ASSIGNED
Version: unspecified → 17
Comment on attachment 673638 [details] [diff] [review] patch I'm going to say ui-r-, because the current patch pushes the "3 items" out past the edge of the list, as https://dl.dropbox.com/u/2301433/Screenshots/ItemsShown.png shows. (It should be where the "0 items" is, because it's the number of items in the list, not the number of buttons… ;) So ui-r-, but feel free to give it a ui-r=me with that fixed. Thanks, Blake.
Attachment #673638 - Flags: ui-review?(bwinton) → ui-review-
OK, fixed that.
Comment on attachment 673993 [details] [diff] [review] patch v2 Review of attachment 673993 [details] [diff] [review]: ----------------------------------------------------------------- r=mkmelin
Attachment #673993 - Flags: review?(mkmelin+mozilla) → review+
Archaeopteryx do you see this as a problem in any locale you can look at? Should it be nominated for TB17?
MXR shows the following translations (but don't trust mxr being up to date, see bug 803810): http://mxr.mozilla.org/l10n-mozilla-aurora/search?string=filterHeader.label Longest translation: /nso/mail/chrome/messenger/FilterListDialog.dtd (View Hg log or Hg annotations) line 18 -- <!ENTITY filterHeader.label "Difilthara tše kgonagaditšwego di sepetšwa ka tsela ya othomethiki ka tatelano yeo e laetšwago ka fase."> Gets shortened at: "[...] othomethiki..." So maybe every fifth translation (I didn't count) translation doesn't show this by default. If you care about polish bugs, uplifting an idea is an option and makes localizers happy. If you don't want to change beta, it's in a resizeable dialog, so the user can work around this.
I think about nominating only because this got broken in TB17, it wrapped fine before. So if we fix it in TB17, it will never get exposed. If we do not fix it in 17, it will be an unnecessary regression until the next release a year after that.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 19.0
(In reply to :aceman from comment #8) > I think about nominating only because this got broken in TB17, it wrapped > fine before. So if we fix it in TB17, it will never get exposed. If we do > not fix it in 17, it will be an unnecessary regression until the next > release a year after that. That's a very good reason for nominating.
Comment on attachment 673993 [details] [diff] [review] patch v2 [Approval Request Comment] Regression caused by (bug #): bug 450302 User impact if declined: comment 8 Testing completed (on c-c, etc.): Risk to taking this patch (and alternatives if risky): Should be none.
You need to log in before you can comment on or make changes to this bug.