1269730 - Firebug theme - Adjust UI of logs view in Console panel

Status: RESOLVED WONTFIX

(DevTools :: General, defect, P2)

Description

[Sebastian Zartner [:sebo]]

When using the Firebug theme the console messages should use the formattings and icons of Firebug 2.

Sebastian

Comment 1

[Sebastian Zartner [:sebo]]

Attachment: Firebug Console in comparison with DevTools' Console panel

Using the Firebug theme the UI of the Console panel still has some differences to the Firebug extension. See the attached screenshot for comparison.
Parts of the differences were noted in the Firebug discussion group[1].

Here's a list of differences, which should be adjusted:

- Background color is slightly darker
- Logged elements have bold tag names and they use different colors
- Source links are not bold and their line and column numbers are red (colors might be ok, though)
- Icons differ
- Groups have a white background and don't have borders around their contents
- Functions in stack are missing their call parameters and are red, source links show the full URL
- XHRs have a white background and are missing the HTTP status, load time and source link

Sebastian

[1] https://groups.google.com/d/msg/firebug/8ICvaq6gFhs/EQSzF2MlAAAJ

Comment 2

[Brian Grinstead [:bgrins]]

Thanks for the comparison

Comment 3

[Tim Nguyen :ntim]

I think we might want to revisit the light/dark theme look to be more similar to Firebug. 
I like the following things from Firebug:
- color-codes the message icons (red for error, yellow for warning, blue for info)
- messages are clearly separated by separators
- the icons are bigger

We might want to take Safari's console as inspiration (which is pretty similar to Firebug).

Helen, what do you think ?

Comment 4

[Sebastian Zartner [:sebo]]

If I'm not mistaken, Helen doesn't work for Mozilla anymore. So I remove the needinfo request for her.

I agree with your points, Tim. I'll try to provide a patch for that.

Sebastian

Comment 5

[Gabriel Luong [:gl] (ΦωΦ)]

(In reply to Sebastian Zartner [:sebo] from comment #4)
> If I'm not mistaken, Helen doesn't work for Mozilla anymore. So I remove the
> needinfo request for her.
> 
> I agree with your points, Tim. I'll try to provide a patch for that.
> 
> Sebastian

There are some design notes here that should be consulted and followed for this patch. 
https://docs.google.com/document/d/1AJ7oHC7akXJiliheHjhFc4fTonaN31Eo_P2M8gssL8s/edit#

Comment 6

[Gabriel Luong [:gl] (ΦωΦ)]

It would be also nice to clarify what the scope of the work will be. There are maybe 2 concerns - does this relate to the devtools photon work and what type of changes. For the latter, I believe we are working on the 3 points that ntim mentioned in comment 3. We should also rename the bug if so and block it on Bug 1389730 if so.

Comment 7

[Sebastian Zartner [:sebo]]

(In reply to Gabriel [:gl] (ΦωΦ) from comment #5)
> There are some design notes here that should be consulted and followed for
> this patch. 
> https://docs.google.com/document/d/
> 1AJ7oHC7akXJiliheHjhFc4fTonaN31Eo_P2M8gssL8s/edit#

Thanks for sharing! I have requested access.

(In reply to Gabriel [:gl] (ΦωΦ) from comment #6)
> It would be also nice to clarify what the scope of the work will be. There
> are maybe 2 concerns - does this relate to the devtools photon work and what
> type of changes. For the latter, I believe we are working on the 3 points
> that ntim mentioned in comment 3.

When I filed this bug it was meant to only adjust the Firebug theme, but one point I heard a few times was that currently log and info messages use the same orange color as error messages, which is confusing. That's one important part I'd like to take in.
So, I'd do the changes of comment 1 (minus the last two points in that list, as they are content changes, which will be put in separate bugs) plus Tim's three changes mentioned in comment 3.

> We should also rename the bug if so and block it on Bug 1389730 if so.

As that are UI changes, they might be part of bug 1389730.

Sebastian

Comment 8

[Jan Honza Odvarko [:Honza] (always need-info? me)]

The Firebug theme was removed in Bug 1378108, so I am marking this report as WONTFIX.

Honza

Comment 9

[Sebastian Zartner [:sebo]]

Actually, this bug also covered changes to the light and dark theme. See comment 3. Asking in bug 878519 whether they can be handled there.

Sebastian