Bug 175274

Summary: Web Inspector: create better icon for Canvas Path navigation item
Product: WebKit Reporter: Devin Rousso <hi>
Component: Web InspectorAssignee: Matt Baker <mattbaker>
Status: RESOLVED FIXED    
Severity: Normal CC: agomez, buildbot, commit-queue, inspector-bugzilla-changes, joepeck, mattbaker, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 175485, 178432    
Attachments:
Description Flags
Patch
none
[Image] Show path button
none
Patch
none
[Image] Show path button - final
none
Patch none

Devin Rousso
Reported 2017-08-07 11:23:15 PDT
Currently, we are using the PaintFlashing icon (paint brush), but that doesn't really convey the idea of a path very well.
Attachments
Patch (2.69 KB, patch)
2017-08-17 13:44 PDT, Matt Baker 		no flags
DetailsFormatted DiffDiff
[Image] Show path button (61.42 KB, image/png)
2017-08-17 13:48 PDT, Matt Baker 		no flags
Details
Patch (2.68 KB, patch)
2017-08-17 18:21 PDT, Matt Baker 		no flags
DetailsFormatted DiffDiff
[Image] Show path button - final (91.95 KB, image/png)
2017-08-17 18:22 PDT, Matt Baker 		no flags
Details
Patch (2.68 KB, patch)
2017-08-17 19:31 PDT, Matt Baker 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Matt Baker
Comment 1 2017-08-17 13:44:45 PDT
Created attachment 318413 [details] Patch
Matt Baker
Comment 2 2017-08-17 13:48:47 PDT
Created attachment 318414 [details] [Image] Show path button I like using the arc, it better conveys "path" than a straight line IMO.
Matt Baker
Comment 3 2017-08-17 18:21:27 PDT
Created attachment 318453 [details] Patch
Matt Baker
Comment 4 2017-08-17 18:22:30 PDT
Created attachment 318454 [details] [Image] Show path button - final Changes based on feedback.
Joseph Pecoraro
Comment 5 2017-08-17 18:51:06 PDT
Comment on attachment 318453 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=318453&action=review Please file a bug for Images/gtk. You may want to copy the PaintFlashing icon as Path.svg for gtk until they have a replacement. It would be bad to have a missing box icon. > Source/WebInspectorUI/UserInterface/Images/Path.svg:6 > + <path stroke="currentColor" fill="none" d="M 3 12 S 3 2 8 8 c 5.44 6.52 5-4 5-4"/> This syntax is a little weird to me. I think the lowercase 'c' is important (relative curve). The "5-4" is this actually "5 -4"? If so a space might be helpful just for readability.
Matt Baker
Comment 6 2017-08-17 19:31:27 PDT
Created attachment 318458 [details] Patch
WebKit Commit Bot
Comment 7 2017-08-17 20:13:57 PDT
Comment on attachment 318458 [details] Patch Clearing flags on attachment: 318458 Committed r220898: <http://trac.webkit.org/changeset/220898>
WebKit Commit Bot
Comment 8 2017-08-17 20:13:58 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 9 2017-08-17 20:14:51 PDT
<rdar://problem/33956146>
Note You need to log in before you can comment on or make changes to this bug.