Bug 142301 - [GTK] Web Inspector: New Images for Console Types
Summary: [GTK] Web Inspector: New Images for Console Types
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Andres Gomez Garcia
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-03-04 12:20 PST by Joseph Pecoraro
Modified: 2015-03-26 18:53 PDT (History)
10 users (show)

See Also:


Attachments
Patch (188.53 KB, patch)
2015-03-26 16:18 PDT, Andres Gomez Garcia
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Pecoraro 2015-03-04 12:20:36 PST
* SUMMARY
My understanding is GTK has their own Inspector Images.

We just added a number of images for Inspector Console Types:

    UserInterface/Images/Eye.svg
    UserInterface/Images/TypeBoolean.svg
    UserInterface/Images/TypeNull.svg
    UserInterface/Images/TypeNumber.svg
    UserInterface/Images/TypeObject.svg
    UserInterface/Images/TypeRegex.svg
    UserInterface/Images/TypeString.svg
    UserInterface/Images/TypeSymbol.svg
    UserInterface/Images/TypeUndefined.svg

So GTK folks will likely want to include images for them.
Comment 1 Joseph Pecoraro 2015-03-04 12:21:27 PST
The New Images were added for:
<https://webkit.org/b/141932> Web Inspector: New Object Tree View UI
Comment 2 Radar WebKit Bug Importer 2015-03-04 12:21:28 PST
<rdar://problem/20042627>
Comment 3 Carlos Garcia Campos 2015-03-04 23:16:33 PST
(In reply to comment #0)
> * SUMMARY
> My understanding is GTK has their own Inspector Images.

Yes.

> We just added a number of images for Inspector Console Types:
> 
>     UserInterface/Images/Eye.svg
>     UserInterface/Images/TypeBoolean.svg
>     UserInterface/Images/TypeNull.svg
>     UserInterface/Images/TypeNumber.svg
>     UserInterface/Images/TypeObject.svg
>     UserInterface/Images/TypeRegex.svg
>     UserInterface/Images/TypeString.svg
>     UserInterface/Images/TypeSymbol.svg
>     UserInterface/Images/TypeUndefined.svg
> 
> So GTK folks will likely want to include images for them.

Thank you for let us know.
Comment 4 Andres Gomez Garcia 2015-03-26 16:18:26 PDT
Created attachment 249535 [details]
Patch
Comment 5 Joseph Pecoraro 2015-03-26 16:25:27 PDT
Comment on attachment 249535 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=249535&action=review

rs=me!

- Note we're in the process of considering a new icon (Pencil) for Setters as a corollary for the Eye for Getters. I'll keep you informed if we do.

I'll file bugs for you.

> Source/WebInspectorUI/UserInterface/Images/gtk/ReloadFull.svg:1
> +<?xml version="1.0" encoding="UTF-8" standalone="no"?>

Oh nice! You found the ReloadFull.svg I just added. I forgot to update this bug.

> Source/WebInspectorUI/UserInterface/Images/gtk/ReloadFull.svg:14
> +<!-- Created with Inkscape (http://www.inkscape.org/) -->
> +
> +<svg
> +   xmlns:dc="http://purl.org/dc/elements/1.1/"
> +   xmlns:cc="http://creativecommons.org/ns#"
> +   xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
> +   xmlns:svg="http://www.w3.org/2000/svg"
> +   xmlns="http://www.w3.org/2000/svg"
> +   version="1.1"
> +   width="13"
> +   height="13"
> +   id="svg5505">
> +  <defs

You may want to consider stripping unnecessary attributes / comments / verbosity from these SVGs. I have a feeling some of the attributes can be eliminated.
Comment 6 Andres Gomez Garcia 2015-03-26 18:05:01 PDT
(In reply to comment #5)
> - Note we're in the process of considering a new icon (Pencil) for Setters
> as a corollary for the Eye for Getters. I'll keep you informed if we do.
> 
> I'll file bugs for you.

Great! Really appreciated :)

> > Source/WebInspectorUI/UserInterface/Images/gtk/ReloadFull.svg:14
> > +<!-- Created with Inkscape (http://www.inkscape.org/) -->
> > +
> > +<svg
> > +   xmlns:dc="http://purl.org/dc/elements/1.1/"
> > +   xmlns:cc="http://creativecommons.org/ns#"
> > +   xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
> > +   xmlns:svg="http://www.w3.org/2000/svg"
> > +   xmlns="http://www.w3.org/2000/svg"
> > +   version="1.1"
> > +   width="13"
> > +   height="13"
> > +   id="svg5505">
> > +  <defs
> 
> You may want to consider stripping unnecessary attributes / comments /
> verbosity from these SVGs. I have a feeling some of the attributes can be
> eliminated.

Yeah, I've been thinking about it. Not sure about stripping the information about the license but, definitively, there is quite some unneeded attributes, including gradients Inkscape refuses to remove even if no items are using them in the current document.

I have to find the time to look for a nice way to do this without having to go one icon after another.

Thanks for the heads up anyway.
Comment 7 WebKit Commit Bot 2015-03-26 18:53:52 PDT
Comment on attachment 249535 [details]
Patch

Clearing flags on attachment: 249535

Committed r182048: <http://trac.webkit.org/changeset/182048>
Comment 8 WebKit Commit Bot 2015-03-26 18:53:57 PDT
All reviewed patches have been landed.  Closing bug.