1118384 - Move the new logviewer icon to webapp/app/img

Status: VERIFIED FIXED

(Tree Management :: Treeherder, defect, P1)

Description

[Jonathan French (:jfrench)]

In the course of fixing bug 1057349, I created a variant .svg icon licensed under Creative Commons from http://flaticon.com.

I originally put it in a new folder, webapp/app/vendor/img/. Mauro would prefer it live with all the other icons in webapp/app/img/ so we don't need to add another entry to the manifest during grunt builds and pushes to the instances.

So this change will do that move, assuming everyone is ok with CC licensed icons co-mingling with our own. I am not sure how we will differentiate between the two types in the dir going forward though, other than I suppose tracing their git history.

We will still keep the CC footer in Help, which refers to use of CC icons.

Comment 1

[Ed Morley [:emorley]]

P1 since stage gets:
GET https://treeherder.allizom.org/vendor/img/logviewerIcon.svg [HTTP/1.1 404 NOT FOUND 172ms]

Comment 2

[Jonathan French (:jfrench)]

Attachment: treeherder-ui-PR#329

Please see above PR for status and review.

Comment 3

[Treeherder GitHub Bugbot]

Commits pushed to master at https://github.com/mozilla/treeherder-ui

https://github.com/mozilla/treeherder-ui/commit/07035ac49cf73bbdb679392f76b3c393ab58abbe
Bug 1118384 - Move the new logviewer icon to webapp/app/img

https://github.com/mozilla/treeherder-ui/commit/468ad3d936fcd6c1c3e974b5f1117608d4ed300a
Merge pull request #329 from tojonmz/move-logviewericon

Bug 1118384 - Move the new logviewer icon to webapp/app/img

Comment 4

[Jonathan French (:jfrench)]

Verified fixed on stage.

Comment 5

[Treeherder GitHub Bugbot]

Commits pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/871e6c34f2dbcb29e44ec30de19e1c3234ffff00
Bug 1118384 - Move the new logviewer icon to webapp/app/img

https://github.com/mozilla/treeherder/commit/caf2bb22d5316f52d0b1b449516755687a2028d8
Merge pull request #329 from tojonmz/move-logviewericon

Bug 1118384 - Move the new logviewer icon to webapp/app/img