Bug 98701

Summary: Web Inspector: Zebra stripe the console
Product: WebKit Reporter: johnjbarton <johnjbarton>
Component: Web Inspector (Deprecated)Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, bweinstein, dubroy, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, webkit.review.bot, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
screenshot of zebra stripes
none
Patch none

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.