1410745 - Remove spacing between attachmentitems on Windows default theme

Status: RESOLVED FIXED

(Thunderbird :: Theme, enhancement)

Description

[Richard Marti (:Paenglab)]

For bug 229224 we need to remove the spacing between the attachmentitems on Windows default theme

Comment 1

[Richard Marti (:Paenglab)]

Attachment: attachment-style.patch

This removes the gap between the attachmentitems and looks now because of this more like a Explorer list.

I also added a 1px padding in attacjmentBucket to make the focus highlight better visible.

The many -moz-border-* changes in attachmentList.css are white space changes to make the rgba() notations consistent. The last change in this file is to make the borders not too strong looking when two or more attachmentitems are touching and are selected.

Comment 2

[Jorg K (CEST = GMT+2)]

I assume we're talking about the attachments in the attachment bucket in a new e-mail.

On Windows 7 I see no difference compared to a Daily which is a few days old. "Gap between items" has changed?

Comment 3

[Richard Marti (:Paenglab)]

Yes, there is a 1px gap between the items without the patch that leads to the issue Thomas noted in bug 229224 when dragging them over other items. I can make a screenshot this evening.

Comment 4

[Jorg K (CEST = GMT+2)]

Comment on attachment 8921613 [details] [diff] [review]
attachment-style.patch

OK, I can see the difference now when attaching 10+ items. Before I only added three.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/260d49e11c29
Fix spacing between attachment items in attachment bucket. r=jorgk

Comment 6

[Thomas D. (:thomas8)]

Richard, please remove the focus border from attachment listbox (big square). At first I thought it's a bug.
- apart from visual noise, it does not add anything, because there's a clear and non-ambiguous focus indication on the focused item: special border and darker blue background if focused and selected.
- it is technically misleading/incorrect/confusing, because focus is NOT on the bucket, but on the focused item inside the bucket.
- it is needlessly deviating from platform focus standards, and our own standards in other listboxes like contacts side bar
(- it is incomplete: missing border on the right)

Comment 7

[Thomas D. (:thomas8)]

I just realize that it's better than before, but it's still not correct/perfect.

Please try to make it exactly as in windows Explorer, which is almost exactly as in contacts sidebar:
- selected, but not focused: no border on left and right, inside border with weaker border-color, no outside border at selection edge (no border-top of top item/no border-bottom of bottom item of selection).
- selected and focused: 4-sides border, emphasized with stronger border-color

Contacts sidebar is almost there, but two minor flaws:
- selected & focused item has stronger top border, which isn't intended I think
- last selected item has bottom border and shouldn't

After that, we can test/discuss if there's any way to emphasize the 4-side focus border more, I guess just making it a bit darker would suffice (2px might look odd and cause jumping problems).

Comment 8

[Thomas D. (:thomas8)]

And please use exactly same colors as for contacts side bar. I wonder if we don't have standardized colors for that, instead of hard-coding everywhere?

Comment 9

[Richard Marti (:Paenglab)]

This bug is fixed. Further work needs a new bug.

(In reply to Thomas D. (currently busy elsewhere) from comment #7)
> I just realize that it's better than before, but it's still not
> correct/perfect.
> 
> Please try to make it exactly as in windows Explorer, which is almost
> exactly as in contacts sidebar:
> - selected, but not focused: no border on left and right, inside border with
> weaker border-color, no outside border at selection edge (no border-top of
> top item/no border-bottom of bottom item of selection).

What for a strange Explorer do you have. I see always borders on all sides

> - selected and focused: 4-sides border, emphasized with stronger border-color
> 
> Contacts sidebar is almost there, but two minor flaws:
> - selected & focused item has stronger top border, which isn't intended I
> think
> - last selected item has bottom border and shouldn't

Again, my Explorer has always all four borders. Please show screenshots, but not in this bug.

> After that, we can test/discuss if there's any way to emphasize the 4-side
> focus border more, I guess just making it a bit darker would suffice (2px
> might look odd and cause jumping problems).

Can be done.