Bug 98701 - Web Inspector: Zebra stripe the console
Summary: Web Inspector: Zebra stripe the console
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-08 16:02 PDT by johnjbarton
Modified: 2012-10-11 06:56 PDT (History)
12 users (show)

See Also:


Attachments
Patch (1.34 KB, patch)
2012-10-09 11:11 PDT, johnjbarton
no flags Details | Formatted Diff | Diff
screenshot of zebra stripes (30.70 KB, image/png)
2012-10-09 11:17 PDT, johnjbarton
no flags Details
Patch (1.31 KB, patch)
2012-10-10 09:21 PDT, johnjbarton
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description johnjbarton 2012-10-08 16:02:01 PDT
I find it hard to hit the links on the RHS of the Console panel. I think zebra striping would help, and now-a-days its super easy.

index 04ada32..3a19113 100644
--- a/Source/WebCore/inspector/front-end/inspector.css
+++ b/Source/WebCore/inspector/front-end/inspector.css
@@ -1020,6 +1020,10 @@ body.platform-linux .monospace, body.platform-linux .source-code {
     color: rgb(15%, 15%, 15%);
 }
 
+.console-message:nth-of-type(even) { 
+    background-color: #F0F0F0;
+}
+
 .console-message-url {
     float: right;
     margin-left: 4px;
Comment 1 johnjbarton 2012-10-09 11:11:16 PDT
Created attachment 167790 [details]
Patch
Comment 2 johnjbarton 2012-10-09 11:17:38 PDT
Created attachment 167793 [details]
screenshot of zebra stripes
Comment 3 Pavel Feldman 2012-10-10 07:09:04 PDT
I must admin I don't like it. Adding Patrick.
Comment 4 Pavel Feldman 2012-10-10 07:09:13 PDT
*admit
Comment 5 dubroy 2012-10-10 07:30:31 PDT
I'm not a big fan of permanent zebra-striping either. What about showing a row highlight on hover, like we do in the Elements pane? An even more subtle technique would be to only show the highlight when you are hovering over a link on the RHS.
Comment 6 johnjbarton 2012-10-10 09:21:26 PDT
Created attachment 168012 [details]
Patch
Comment 7 dubroy 2012-10-11 06:52:37 PDT
Why #F0F0F0? It should probably be the same color as the hover color the Elements pane, for consistency.
Comment 8 WebKit Review Bot 2012-10-11 06:56:49 PDT
Comment on attachment 168012 [details]
Patch

Clearing flags on attachment: 168012

Committed r131053: <http://trac.webkit.org/changeset/131053>
Comment 9 WebKit Review Bot 2012-10-11 06:56:53 PDT
All reviewed patches have been landed.  Closing bug.