803916 - filterHeader.label label in filterListDialog no longer wraps after bug 450302

Status: RESOLVED FIXED

(Thunderbird :: Filters, defect)

Description

[:aceman]

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).

Comment 1

[:aceman]

Attachment: patch

Comment 2

[:aceman]

This may have a bigger impact on languages that have that string longer than the en-US locale.

Comment 3

[Blake Winton (:bwinton) (:☕️)]

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.

Comment 4

[:aceman]

Attachment: patch v2

OK, fixed that.

Comment 5

[Magnus Melin [:mkmelin]]

Comment on attachment 673993 [details] [diff] [review]
patch v2

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

r=mkmelin

Comment 6

[:aceman]

Archaeopteryx do you see this as a problem in any locale you can look at? Should it be nominated for TB17?

Comment 7

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

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.

Comment 8

[:aceman]

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.

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/comm-central/rev/451f5e07043e

Comment 10

[Mark Banner (:standard8)]

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

[:aceman]

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.

Comment 12

[Mark Banner (:standard8)]

https://hg.mozilla.org/releases/comm-aurora/rev/f55fcf3d913f
https://hg.mozilla.org/releases/comm-beta/rev/bc4161c7ee5c