Closed Bug 1454486 Opened 2 years ago Closed 2 years ago
We don't think people realize the suitcase is ebay. Replacing with the word eBay. [Tracking Requested - why for this release]: Trying to increase eBay use.
Comment on attachment 8968343 [details] Bug 1454486 - Update eBay icon. https://reviewboard.mozilla.org/r/237020/#review242874 Image doesn't open for me locally after I import your patch from mozreview. I tried dragging it to Firefox, and it resulted in a beachball (had to kill the browser completely).
Attachment #8968343 - Flags: review?(francesco.lodolo) → review-
Interestingly enough, Gimp on Linux manages to load the file, and export a working image.
I always forget about the icon thing. > I tried dragging it to Firefox, and it resulted in a beachball (had to kill the browser completely). That seems bad. I wonder if that happens when it loads from a web page?
Comment on attachment 8968343 [details] Bug 1454486 - Update eBay icon. https://reviewboard.mozilla.org/r/237020/#review243046
Attachment #8968343 - Flags: review?(francesco.lodolo) → review+
(In reply to Mike Kaply [:mkaply] from comment #6) > I always forget about the icon thing. > > > I tried dragging it to Firefox, and it resulted in a beachball (had to kill the browser completely). > > That seems bad. I wonder if that happens when it loads from a web page? I don't think so. I tried converting into a data URI via http://freeonlinetools24.com/base64-image And that loads when pasted in the address bar. I've also tried again with beta, and it worked, not sure if it was a one-time issue.
Comment on attachment 8968343 [details] Bug 1454486 - Update eBay icon. Approval Request Comment [Feature/Bug causing the regression]: Update eBay icon [User impact if declined]: eBay icon is not recognizable [Is this code covered by automated tests?]: No [Has the fix been verified in Nightly?]: Not yet [Needs manual test from QE? If yes, steps to reproduce]: No [List of other uplifts needed for the feature/fix]: No [Is the change risky?]: No [Why is the change risky/not risky?]: Just an icon change [String changes made/needed]: None We believe the eBay suitcase icon isn't recognized by users, so we're moving to an older icon that has eBay in it.
Attachment #8968343 - Flags: approval-mozilla-beta?
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/bbe44fc94f51 Update eBay icon. r=flod
Comment on attachment 8968343 [details] Bug 1454486 - Update eBay icon. icon change, approved for 60.0b14
Attachment #8968343 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
(In reply to Mike Kaply [:mkaply] from comment #0) > ... Trying to increase eBay use. Just curious on the why for this one.
> ... Trying to increase eBay use. We have partner relationship with eBay, so we'd love to see it used more. But it's really about making people aware that we've integrated eBay again.
You need to log in before you can comment on or make changes to this bug.