712159 - Line between window title bar and toolbar in standalone window

Status: RESOLVED FIXED

(Thunderbird :: Theme, defect)

Description

[Andreas Nilsson (:andreasn)]

Attachment: screenshot of the issue

I think this is restricted to the tabs on top patch (bug #644169)
There seems to be a line between the window title bar and the toolbar when you open a message in a new window.

Comment 1

[Andreas Nilsson (:andreasn)]

It seems that the only thing I can see differ between standalone window and and the 3pane window is the nodeName mail:messageWindow vs. mail:3pane.

I tried to select this with:
window[windowtype=mail:messageWindow] .mail-toolbox {
  -moz-appearance: toolbox;
}

The css parser did not enjoy that at all.
Then again, it's 3:30 in the morning, so I might be missing something. :)

Comment 2

[Jim Porter (:squib)]

I think you need quotes around "mail:messageWindow".

Comment 3

[Andreas Nilsson (:andreasn)]

Attachment: patch fixing the issue

The bug is caused by the special toolbar styling we're doing to the main toolbar in 3pane. This patch special-cases the standalone window case.

Comment 4

[Andreas Nilsson (:andreasn)]

Attachment: screenshot of patch in action

Comment 5

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

Comment on attachment 583185 [details] [diff] [review]
patch fixing the issue

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

Code looks good (modulo the notes below).  r=me.
And the screenshot you posted looks good, except that the lines are missing from the toolbars in the 3-pane, too…  So, uh, I'm not sure what to put for ui-r…

I'll say ui-r=me if the main window still has lines between the toolbars.

::: mail/themes/pinstripe/mail/primaryToolbar.css
@@ -133,4 +141,4 @@
> >    margin: 5px 4px 3px;
> >  }
> >  
> > -#tabbar-toolbar .toolbarbutton-1 {
> > +#tabbar-toolbar > .toolbarbutton-1 {

Is this change (and the previous one) needed to fix the bug, or is it just cleanup suggested by Sid?

@@ -139,4 +147,5 @@
> >    padding-top: 0;
> >    padding-bottom: 0;
> >  }
> >  
> > +

Could we remove this extra line?

Comment 6

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

Huh.  As Andreas pointed out on IRC, we don't have the lines between the toolbars in Windows or Linux, or in Firefox on OSX, so removing them here makes sense, too.

Given that, ui-r=me just the way it is.

Thanks,
Blake.

Comment 7

[Mike Conley (:mconley) (:⚙️)]

Checked in to comm-central as http://hg.mozilla.org/comm-central/rev/0dd28e137266