The default bug view has changed. See this FAQ.

not possible to see if the left pane of the Chat tab has focus if "Conversations" is selected on Windows/Linux

RESOLVED FIXED in Thunderbird 15.0

Status

Thunderbird
Theme
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: florian, Assigned: andreasn)

Tracking

unspecified
Thunderbird 15.0
x86_64
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

5 years ago
Created attachment 624788 [details] [diff] [review]
Patch

Jb noticed that on Linux it's impossible to see if the left pane (the listbox) of the Chat tab has focus when the "Conversations" item is selected.
I looked at the code and it seems that's broken on Windows too.

The attached patch should make that list look the list of folders of the Mail 3pane view for Windows and Linux.

This bug makes the changes I did in bug 740765 difficult to understand/discover.
Attachment #624788 - Flags: review?(nisses.mail)
(Assignee)

Comment 1

5 years ago
Comment on attachment 624788 [details] [diff] [review]
Patch

diff -r 499441818d84 mail/themes/gnomestripe/mail/chat.css
+:-moz-any(imconv, imcontact, imgroup)[selected] {
+  background-color: -moz-cellhighlight;
+}

This also wants a:
color: -moz-cellhighlighttext;

+  border: 1px dotted #F3D982;
How come this color value?
(same goes for qute)

diff -r 499441818d84 mail/themes/qute/mail/chat.css
+:-moz-any(imconv, imcontact, imgroup)[selected] {
+  background-color: -moz-cellhighlight;
+}

This seems to break the style under Aero, so the corresponding selectors needs to be added to chat-aero.css
Attachment #624788 - Flags: review?(nisses.mail) → review-
(Reporter)

Comment 2

5 years ago
(In reply to Andreas Nilsson (:andreasn) from comment #1)

> +  border: 1px dotted #F3D982;
> How come this color value?
> (same goes for qute)

http://mxr.mozilla.org/comm-central/source/mozilla/toolkit/themes/winstripe/global/tree.css#56
http://mxr.mozilla.org/comm-central/source/mozilla/toolkit/themes/gnomestripe/global/tree.css#60

> 
> diff -r 499441818d84 mail/themes/qute/mail/chat.css
> +:-moz-any(imconv, imcontact, imgroup)[selected] {
> +  background-color: -moz-cellhighlight;
> +}
> 
> This seems to break the style under Aero, so the corresponding selectors
> needs to be added to chat-aero.css

I'm not sure of how the rules at http://mxr.mozilla.org/comm-central/source/mail/themes/qute/mail/chat-aero.css#92 and the next 20 or so lines were built, they don't seem to copy the toolkit CSS for trees.
(Assignee)

Comment 3

5 years ago
(In reply to Florian Quèze from comment #2)
> (In reply to Andreas Nilsson (:andreasn) from comment #1)
> 
> > +  border: 1px dotted #F3D982;
> > How come this color value?
> > (same goes for qute)
> 
> http://mxr.mozilla.org/comm-central/source/mozilla/toolkit/themes/winstripe/
> global/tree.css#56
> http://mxr.mozilla.org/comm-central/source/mozilla/toolkit/themes/
> gnomestripe/global/tree.css#60

Ah, thanks!


> > diff -r 499441818d84 mail/themes/qute/mail/chat.css
> > +:-moz-any(imconv, imcontact, imgroup)[selected] {
> > +  background-color: -moz-cellhighlight;
> > +}
> > 
> > This seems to break the style under Aero, so the corresponding selectors
> > needs to be added to chat-aero.css
> 
> I'm not sure of how the rules at
> http://mxr.mozilla.org/comm-central/source/mail/themes/qute/mail/chat-aero.
> css#92 and the next 20 or so lines were built, they don't seem to copy the
> toolkit CSS for trees.

We use the same styling in messenger-aero for the folderpane and messagepane http://mxr.mozilla.org/comm-central/source/mail/themes/qute/mail/messenger-aero.css#74
(Reporter)

Comment 4

5 years ago
(In reply to Andreas Nilsson (:andreasn) from comment #3)

> > > diff -r 499441818d84 mail/themes/qute/mail/chat.css
> > > +:-moz-any(imconv, imcontact, imgroup)[selected] {
> > > +  background-color: -moz-cellhighlight;
> > > +}
> > > 
> > > This seems to break the style under Aero, so the corresponding selectors
> > > needs to be added to chat-aero.css
> > 
> > I'm not sure of how the rules at
> > http://mxr.mozilla.org/comm-central/source/mail/themes/qute/mail/chat-aero.
> > css#92 and the next 20 or so lines were built, they don't seem to copy the
> > toolkit CSS for trees.
> 
> We use the same styling in messenger-aero for the folderpane and messagepane
> http://mxr.mozilla.org/comm-central/source/mail/themes/qute/mail/messenger-
> aero.css#74

I don't see how that code themes differently the focused and non-focused selected item. Is there no difference on aero?

Or do I just need to add "imgroup" in the lists of selectors that contain imconv and imcontact already?
(Assignee)

Comment 5

5 years ago
Yes, I think that should be enough. Will test it once I'm back on windows.
(Reporter)

Comment 6

5 years ago
I had something else to test on my Windows VM so I looked at this too. The folder pane appearance is identical whether it is focused or not (and I think it's a bug). The Windows 7 file explorer uses a blue highlight for selected and focused items, and a gray highlight for selected but without focus.
(Assignee)

Comment 7

5 years ago
That would probably be bug #686959 and Jim said he had a patch to fix that.
(Assignee)

Comment 8

5 years ago
Created attachment 628316 [details] [diff] [review]
fixes win7 aero appearance

This makes sure the patch don't create any regressions for the Aero case. The issue still remains that focused and selected items are identical, but perhaps we can tackle that in #686959 as it needs to be fixed for all other cases (folder view, address book etc.) as well.
(Assignee)

Comment 9

5 years ago
Created attachment 628339 [details] [diff] [review]
and fixes for gnomestripe too

This should do the trick for both qute and gnomestripe.
Attachment #628316 - Attachment is obsolete: true
(Reporter)

Comment 10

5 years ago
Comment on attachment 628339 [details] [diff] [review]
and fixes for gnomestripe too

Given that both Andreas an me have touched that patch for it to reach its current state, it's not clear any more who is the patch author and who is reviewer, so we would appreciate having someone else looking at it. Requesting review from Mike :-).
Attachment #628339 - Flags: review?(mconley)
Comment on attachment 628339 [details] [diff] [review]
and fixes for gnomestripe too

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

From inspection, this looks good - just one super-tiny nit. r=me with that fixed.

::: mail/themes/qute/mail/chat.css
@@ +117,3 @@
>  }
>  
> +

Remove this extra line.
Attachment #628339 - Flags: review?(mconley) → review+
(Reporter)

Comment 12

5 years ago
https://hg.mozilla.org/comm-central/rev/9b6d79b4c4cf
Assignee: nobody → nisses.mail
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 15.0
You need to log in before you can comment on or make changes to this bug.