555452 - [Addressbook] Mail List Dialog header sections should use <hbox align="end"> instead of spacers to align the labels.

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Address Book & Contacts, enhancement)

Description

[Philip Chee]

From Bug 534831 Comment 6

> - abListOverlay.xul sports some odd <spacer flex="1"/> elements which push the
> mailing list header textboxen to the right edge. That looks very weird.

I think that the intention was to right align the labels. We should switch these to a grid and wrap the labels with a <hbox align="end"> instead.

Comment 1

[Philip Chee]

Attachment: Patch v1.0 Move to suite and switch to using a grid.

First time I'm doing a move. make -C $OBJDIR$ tier_apps seems happy with this.

> -             accesskey="&ListName.accesskey;" class="CardEditLabel"/>

No CSS for CardEditLabel was found in comm-central so I removed these from the XUL.

Comment 2

[[e-mail]]

Comment on attachment 436687 [details] [diff] [review]
Patch v1.0 Move to suite and switch to using a grid.

>+      <row>
>+        <hbox align="center" pack="end">
The align would look better on the row.

>+                 class="CardEditWidth"/>
This could go on the column, I think.

(In reply to comment #1)
> No CSS for CardEditLabel was found in comm-central so I removed these from the
> XUL.
Actually we could style them with text-align: right; to avoid the hbox.

Comment 3

[Philip Chee]

Attachment: Patch v1.1 nits fixed.

> (From update of attachment 436687 [details] [diff] [review])
>>+      <row>
>>+        <hbox align="center" pack="end">
> The align would look better on the row.
Fixed.

>>+                 class="CardEditWidth"/>
> This could go on the column, I think.
Fixed.

> (In reply to comment #1)
>> No CSS for CardEditLabel was found in comm-central so I removed these from the
>> XUL.
> Actually we could style them with text-align: right; to avoid the hbox.
Fixed.

Comment 4

[[e-mail]]

Comment on attachment 442654 [details] [diff] [review]
Patch v1.1 nits fixed.

>+      <column class="CardEditWidth"
>+              flex="1"/>
Wasn't necessary to wrap this, it fits on one line.

Comment 5

[Philip Chee]

>>>+      <column class="CardEditWidth"
>>>+              flex="1"/>
> Wasn't necessary to wrap this, it fits on one line.

Fixed locally. Waiting for review.

Comment 6

[Karsten Düsterloh]

Comment on attachment 442654 [details] [diff] [review]
Patch v1.1 nits fixed.

>+++ b/suite/mailnews/addrbook/abListOverlay.xul
>-                 autocompletesearch="addrbook" timeout="300" maxrows="4" 
>+                 autocompletesearch="addrbook" timeout="300" maxrows="4"
>                  completedefaultindex="true" forcecomplete="true"

Transform these two lines to "one line per attribute" as well.

r=me with that.

(The listbox in these dialogs is a true beauty: a fixed height plus predefined listitems for the visible area. Spending a flex made me admire your well-considered bug scope. *g*)

Comment 7

[Philip Chee]

Checked-in
http://hg.mozilla.org/comm-central/rev/23337051ed93