966661 - Devtools themes - Light theme issues

Status: RESOLVED FIXED

(DevTools :: Inspector, defect)

Description

[Tim Nguyen :ntim]

Attachment: Screenshot

Here's a list :
- sidemenu arrow is missing (I actually know why, it's because the file is named itemArrow-ltr.png while it should be itemArrow-ltr.svg, as it's an svg file)
- The addon command buttons are inverted
- The search icon is too light and nearly invisible
- Infobar is still dark

These are details and are less important :
- The tab active effect is horrible
- Some borders are too dark compared to the mockup

Comment 1

[Tim Nguyen :ntim]

Other bugs in sidebar dock mode :
- Console, Inspector and network "side"bars have a black top border

Comment 2

[Brian Grinstead [:bgrins]]

> - sidemenu arrow is missing (I actually know why, it's because the file is
> named itemArrow-ltr.png while it should be itemArrow-ltr.svg, as it's an svg
> file)

Opened Bug 967033.

> - The addon command buttons are inverted

Yes, I've noticed this too.  I suppose we should not invert them, though if the addon author has uploaded a white icon because of the historically dark-only toolbars, it would not be visible.

> - The search icon is too light and nearly invisible

Bug 966907

> - Infobar is still dark

Since this is overlaid on web content (and not part of the normal devtools chrome), I think a case could be made for leaving it as dark.  I do see the point of using similar colors for consistency though.  There is also a bit of a challenge here - the main browser window (on which this is overlaid) is not aware of which devtools theme is applied.

Comment 3

[Tim Nguyen :ntim]

(In reply to Brian Grinstead [:bgrins] from comment #2)

> > - The addon command buttons are inverted
> 
> Yes, I've noticed this too.  I suppose we should not invert them, though if
> the addon author has uploaded a white icon because of the historically
> dark-only toolbars, it would not be visible.
> 
As you can see in the screenshot, the IE, Opera, Chrome icons are inverted, that the addon can't actually provide icons for them (it just takes the browser icon). Yes, if the author provides a white icon it can be a good idea to invert it, but no addon does it. I guess a good solution would be extend the toolbox tab functionality adding icon-light as parameter. (Same for the command icons, which are affected) This would take some time, but at least addons will not be affected.


> > - Infobar is still dark
> 
> Since this is overlaid on web content (and not part of the normal devtools
> chrome), I think a case could be made for leaving it as dark.  I do see the
> point of using similar colors for consistency though.  There is also a bit
> of a challenge here - the main browser window (on which this is overlaid) is
> not aware of which devtools theme is applied.

I think this is not high priority, but I think it's a good idea to make it light to match the mockup.

Comment 4

[Brian Grinstead [:bgrins]]

(In reply to Tim Nguyen [:ntim] from comment #3)
> (In reply to Brian Grinstead [:bgrins] from comment #2)
> 
> > > - The addon command buttons are inverted
> > 
> > Yes, I've noticed this too.  I suppose we should not invert them, though if
> > the addon author has uploaded a white icon because of the historically
> > dark-only toolbars, it would not be visible.
> > 
> As you can see in the screenshot, the IE, Opera, Chrome icons are inverted,
> that the addon can't actually provide icons for them (it just takes the
> browser icon). Yes, if the author provides a white icon it can be a good
> idea to invert it, but no addon does it. I guess a good solution would be
> extend the toolbox tab functionality adding icon-light as parameter. (Same
> for the command icons, which are affected) This would take some time, but at
> least addons will not be affected.

Fair enough, I have opened Bug 967168 to not invert these icons

Comment 5

[Brian Grinstead [:bgrins]]

(In reply to Tim Nguyen [:ntim] from comment #3)
> (In reply to Brian Grinstead [:bgrins] from comment #2)
> 
> > > - The addon command buttons are inverted
> > 
> > Yes, I've noticed this too.  I suppose we should not invert them, though if
> > the addon author has uploaded a white icon because of the historically
> > dark-only toolbars, it would not be visible.
> > 
> As you can see in the screenshot, the IE, Opera, Chrome icons are inverted,
> that the addon can't actually provide icons for them (it just takes the
> browser icon). Yes, if the author provides a white icon it can be a good
> idea to invert it, but no addon does it. I guess a good solution would be
> extend the toolbox tab functionality adding icon-light as parameter. (Same
> for the command icons, which are affected) This would take some time, but at
> least addons will not be affected.
> 

By the way, what addons are you using that add the the IE, Opera and Chrome command icons?

Comment 6

[Tim Nguyen :ntim]

(In reply to Brian Grinstead [:bgrins] from comment #5)
> (In reply to Tim Nguyen [:ntim] from comment #3)
> > (In reply to Brian Grinstead [:bgrins] from comment #2)
> > 
> > > > - The addon command buttons are inverted
> > > 
> > > Yes, I've noticed this too.  I suppose we should not invert them, though if
> > > the addon author has uploaded a white icon because of the historically
> > > dark-only toolbars, it would not be visible.
> > > 
> > As you can see in the screenshot, the IE, Opera, Chrome icons are inverted,
> > that the addon can't actually provide icons for them (it just takes the
> > browser icon). Yes, if the author provides a white icon it can be a good
> > idea to invert it, but no addon does it. I guess a good solution would be
> > extend the toolbox tab functionality adding icon-light as parameter. (Same
> > for the command icons, which are affected) This would take some time, but at
> > least addons will not be affected.
> > 
> 
> By the way, what addons are you using that add the the IE, Opera and Chrome
> command icons?

https://addons.mozilla.org/en-us/firefox/addon/open-with/

Comment 7

[Tim Nguyen :ntim]

Oh, btw, the developer toolbar is still dark too.