Remove unused wait cursor assets from ui/img

RESOLVED FIXED

Status

P3
normal
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jfrench, Assigned: jfrench)

Tracking

Details

Attachments

(1 attachment)

46 bytes, text/x-github-pull-request
wlach
: review+
Details | Review | Splinter Review
(Assignee)

Description

3 years ago
We have a bunch of unused wait cursor assets in ui/img, and we now use bootstrap's fa-spinner for our wait cursor, so let's remove them.

https://github.com/mozilla/treeherder/blob/afe16b0c000296877595706fc000a074f8d9e8e5/ui/css/treeherder.css#L828-L830

In keeping with our goal for more modest PR requests, I'm intentionally doing this outside the scope of the larger splitting changes in bug 1193804.
(Assignee)

Updated

3 years ago
Assignee: nobody → tojonmz
Status: NEW → ASSIGNED
(Assignee)

Comment 1

3 years ago
Created attachment 8664207 [details] [review]
PR 987

Tweak PR for review.

nb. I misstated the above comment 0, glyphicon halflings is bootstrap's native icon set :)
Attachment #8664207 - Flags: review?(wlachance)
Comment on attachment 8664207 [details] [review]
PR 987

lgtm!
Attachment #8664207 - Flags: review?(wlachance) → review+
(Assignee)

Comment 4

3 years ago
Marking fixed per above merge.
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.