763821 - Error console looks wonky on OS X

Status: RESOLVED FIXED

(Toolkit Graveyard :: Error Console, defect)

Description

[Justin Dolske [:Dolske]]

Attachment: Screenshot w/ new + old console

Bug 760951 added a filter box to the error console, but made the appearance wonky on OS X... OS X convention is that the section "tabs" (all,errors,warning,etc -- not sure of the proper widget name) should be centered in the window. The addition of the filter box pushed them flush-left, which looks out of place.

Not sure what the solution is here. Re-centering the "tabs" would be a little tricky in XUL, and might still look odd with the filter field crowding it. Closest UI I can find is either a Finder window (which achieves balance by having navigation buttons on on the left), or Safari's own error console which uses a different widget style for the all/errors/warnings selection.

Comment 1

[Tom S [:evilpie]]

Sorry, I won't be of much help here, this was the first time I looked at XUL and I don't own an OS X system.

Comment 2

[[e-mail]]

Move the filter box to ToolbarEval perhaps?

Comment 3

[[e-mail]]

Attachment: Proof of concept

Comment 4

[Lukas Blakk [:lsblakk] use ?needinfo]

Gavin, this is not something we would hold a release for - seems like a nice-to-have but it does work in it's current state. If you can get someone assigned to working on this we'll continue tracking but otherwise I'm inclined to untrack this based on lack of activity and a lack of serious user effect.

Comment 5

[:Gavin Sharp [email: [e-mail]]]

I disagree; I think it would be ridiculous to ship this as-is. The solutions aren't hard, we just need to pick one (either back out the patch that caused this, or take something like Neil's patch). I'll make sure that happens.

Comment 6

[:Gavin Sharp [email: [e-mail]]]

I backed bug 760591 out on Aurora for Firefox 16:
https://hg.mozilla.org/releases/mozilla-aurora/rev/5e6da3c55e7c

Comment 7

[Jared Wein [:jaws] (please needinfo? me)]

That previous comment should have said "... backed out bug 760951 ..." (note the typo in the bug number). The commit message had the correct bug number.

Comment 8

[Tom S [:evilpie]]

Well can we take Neil's patch for this cycle?

Comment 9

[:Gavin Sharp [email: [e-mail]]]

Does it work? Can you take this bug? If not we'll need to just keep backing this patch out.

Comment 10

[Jared Wein [:jaws] (please needinfo? me)]

I talked with Tom at MozCamp and we think that a filter toolbar, similar to the find bar for the browser, will work here and won't make any of the platforms look awkward.

Tom, would you like to work on this?

Comment 11

[Tom S [:evilpie]]

Yep I am looking into this.

Comment 12

[Tom S [:evilpie]]

Attachment: filter at the bottom

How would you like this? http://imgur.com/RNjNU
Would appreciate somebody testing this on Mac this time :)

Comment 13

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 660057 [details] [diff] [review]
filter at the bottom

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

I tested this out on OS X and it looks good to me. I think this is a reasonable compromise that will allow us to ship this feature.

Comment 14

[Tom S [:evilpie]]

So should I land this?

Comment 15

[Dave Townsend [:mossop]]

Comment on attachment 660057 [details] [diff] [review]
filter at the bottom

This looks pretty good on windows now and sort of weird with it moved to the bottom, I don't want to make a change that affects all platforms just to fix one without at least checking if UX are ok with this. Screenshot in comment 12.

Comment 16

[[e-mail]]

(In reply to Tom Schuster from comment #12)
> How would you like this? http://imgur.com/RNjNU

Bugzilla accepts image attachments too, you know ;-)

Comment 17

[Lukas Blakk [:lsblakk] use ?needinfo]

(In reply to :Gavin Sharp (use [e-mail] for email) from comment #9)
> Does it work? Can you take this bug? If not we'll need to just keep backing
> this patch out.

Given that this is blocked on UI review can someone nominate and backout on Aurora 17 before we go to Beta channel?

Comment 18

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 660057 [details] [diff] [review]
filter at the bottom

I don't think we need to block on ui-review, this is hidden-by-default UI.

Comment 19

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/71c76fb26e9e

Comment 20

[:Gavin Sharp [email: [e-mail]]]

https://hg.mozilla.org/mozilla-central/rev/71c76fb26e9e

Comment 21

[Dão Gottwald [:dao]]

(In reply to Dave Townsend (:Mossop) from comment #15)
> Comment on attachment 660057 [details] [diff] [review]
> filter at the bottom
> 
> This looks pretty good on windows now and sort of weird with it moved to the
> bottom, I don't want to make a change that affects all platforms just to fix
> one without at least checking if UX are ok with this. Screenshot in comment
> 12.

One way to make it look less weird would be to use the standard widget for this type of UI, <statusbar>, rather than a plain hbox.

I was close to backing this out since it landed without toolkit peer review, but filed bug 799081 instead since the problem this solved on OS X is worse. (However since bug 760951 went through the same broken process and given bug 760951 comment 12 and below, maybe I should have backed out both patches.)

Comment 22

[Willy_ Foo_Foo]

Sorry but why has been the error console menu entry

web developer-error console removed?

Comment 23

[Jared Wein [:jaws] (please needinfo? me)]

(In reply to Sillius Soddus from comment #22)
> Sorry but why has been the error console menu entry
> 
> web developer-error console removed?

See https://bugzilla.mozilla.org/show_bug.cgi?id=602006

Comment 24

[Lukas Blakk [:lsblakk] use ?needinfo]

This is a regression tracked for 17 so an uplift nomination would be great here.

Comment 25

[Jared Wein [:jaws] (please needinfo? me)]

Bug 760951 was backed out of Firefox 17. In the style of comment #6, I've marked this as fixed for fx17.