Bug 29309 - new icon needed for Watch Expressions delete button
Summary: new icon needed for Watch Expressions delete button
Status: RESOLVED WONTFIX
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: 2009-09-16 13:33 PDT by Patrick Mueller
Modified: 2012-10-15 00:52 PDT (History)
3 users (show)

See Also:


Attachments
image showing icon (36.50 KB, image/png)
2009-09-16 13:33 PDT, Patrick Mueller
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Mueller 2009-09-16 13:33:47 PDT
Created attachment 39659 [details]
image showing icon

Attached is an image with an orange arrow pointing to the horrifying red delete icons used in the Watch Expressions pane.  I'm to blame; I just reused an existing icon.  

My opinions: on the plus side, the size is perfect - I think about as big as you can get them to fit on these lines with the current font used.  On the minus side, too red, and the X-ish-ness isn't all that apparent.  Complete guess that perhaps a grey or more subdued red with a single / instead of the x, would look better.  The single / might stand out more than the two strokes required for the X.

Here's the current CSS used in inspector.css for the icon:

/* FIXME: need a better icon (comment in bug 27514) */
.section .properties .delete-button {
    width: 10px;
    height: 10px;
    background-image: url(Images/errorIcon.png);
    background-position: 0 0;
    background-color: transparent;
    background-repeat: no-repeat;
    border: 0 none transparent;
}
Comment 1 Yury Semikhatsky 2012-10-15 00:52:13 PDT
Looks obsolete.