585566 - “Looking at” styling broken

Status: RESOLVED FIXED

(DevTools :: General, defect)

Description

[Joe Walker [:jwalker]]

The “looking at” line in the alternate style view should have a yellow highlight for only the last crumb in the path. The previous crumbs should be the bluey link color used elsewhere.
Similarly the red ‘>>’ should only be inbetween the crumbs, not after each.
Also all but the last crumb should be a link to highlight the element in question

Comment 1

[Joe Walker [:jwalker]]

Also, something I forgot to mention, the links themselves are broken, as you noticed.

Comment 2

[Mihai Sucan [:msucan]]

Attachment: proposed fix

Proposed fix. This patch requires the latest patches from bug 582596.

What I did:
- the crumb only show *between* elements.
- yellow background for the final element in the path.
- blue background for the rest of the path elements. I did it lighter a bit - it was too dark and I couldn't read element names.
- Fixed overflowing of the path. It can now show any depth of path with *no* display issues.
- Now users can click the path elements to jump to element parents. Yay!
- I touched the specificity styling. It wasn't properly aligned.
- I also fixed the "N rules" float on the right, which overflowed other properties when the calculated property value was too long.
- I fixed the table alignment, which was always too large in width. Now horizontal scrollbars do not show for long rule selectors.

What I'd like is to merge this patch into the main patches ASAP. :) It looks/works more polished.

Comment 4

[Mihai Sucan [:msucan]]

Attachment: screenshot

Screenshot for today's meeting with Limi. This image shows the styling changes I did in this patch.

Comment 5

[Mihai Sucan [:msucan]]

Attachment: screenshot 2

A screenshot showing the style panel with a really small width.

Comment 6

[Rob Campbell [:rc] (:robcee)]

do we really need to make the "looking at" element that "flashy"? It stands out a bit too much on the panel. I'd be happy with just lightly-styled text. Code looks decent though.

Comment 7

[Mihai Sucan [:msucan]]

Robert: I know, you have a point. I kept the initial intent of the styling Joe did. The blue path elements had an even darker blue but I decided to lighten it up so I can read the text (I couldn't).

Should I make the blue gradient even brighter?

Comment 8

[Joe Walker [:jwalker]]

I'm including this patch in the main style panel bug: showAll

Comment 9

[Joe Walker [:jwalker]]

cut and paste errorin last comment, sorry:
s/bug: showAll/bug: 582596/