[GTK] Web Inspector: some SVG images are specified 'currentColor' incorrectly
The keyword 'currentColor' is specifed manually in Bug 150602.
But, some SVG images are specified unexpectedly.
And, gtk/NavigationItemCurleyBraces.svg is licensed under NonCommercial CC (Bug 170902).
I'll recreate this image.
Created attachment 307452 [details]
Created attachment 307455 [details]
A screenshot of before and after
Could you please split this into two patches:
* First patch to fix currentColor
* Second patch, in bug #170902, to replace gtk/NavigationItemCurleyBraces.svg with the new version created by you.
Thanks a bunch for fixing these issues. Also thanks for the screenshot, as that makes reviewing it much easier.
Created attachment 307553 [details]
Comment on attachment 307553 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=307553&action=review
> -<!-- Licensed under the Creative Commons Attribution-Share Alike 3.0 United States License (http://creativecommons.org/licenses/by-sa/3.0/) -->
This removes the license that all the others have. Unexpected?
Umm, I should address Bug 170902 before this?
Any idea, Michael?
Please use CC-BY-SA for your new image, yes. And let's land a patch for that image in bug #170902 instead, yes. We shouldn't bury the fix for that in this issue. Perhaps you could upload a new patch here with the changes to just the other two icons.
Created attachment 308333 [details]
Created attachment 308336 [details]
screenshot of before and after this fix
Comment on attachment 308333 [details]
Clearing flags on attachment: 308333
Committed r215867: <http://trac.webkit.org/changeset/215867>
All reviewed patches have been landed. Closing bug.