Closed Bug 812682 Opened 12 years ago Closed 12 years ago

Updated eBay Logos for Desktop

Categories

(Firefox :: Search, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 20
Tracking Status
firefox18 --- verified
firefox19 --- fixed
firefox20 --- verified

People

(Reporter: jnagel, Assigned: ekw)

References

Details

(Whiteboard: [mentor=gavin])

Attachments

(4 files, 2 obsolete files)

Attached image eBay 16x16 icon file (obsolete) (deleted) —
eBay has recently updated their Logo and request that we update the search drop-down with the new logos for Desktop & Mobile
Attached image eBay 16x16 png file (deleted) —
Sorry, not sure how to upload batch files...
Attached image eBay 32x32 png file (deleted) —
This is an easy one to pick-up: just a matter of pushing the 16x16 PNG through http://software.hixie.ch/utilities/cgi/data/data with the "base64" option selected, and then replacing the data: URI in the <Image> in http://mxr.mozilla.org/mozilla-central/source/browser/locales/en-US/searchplugins/eBay.xml with the result. As far as I can tell, we don't have an eBay search for mobile, and bug 795495 isn't fixed yet, so we don't yet have use for the 32x32 version. The 16x16 ICO is a larger file for some reason (does it include both sizes?), so we probably don't need to use that.
Whiteboard: [mentor=gavin]
Attachment #682667 - Attachment is obsolete: true
Summary: Updated eBay Logos for Desktop/Mobile → Updated eBay Logos for Desktop
Attached patch Patch (obsolete) (deleted) — Splinter Review
Assignee: nobody → ewong3
Status: NEW → ASSIGNED
Attachment #684797 - Flags: review?(gavin.sharp)
Comment on attachment 684797 [details] [diff] [review] Patch Thanks, but I forgot to mention one thing - if you copy out of the Firefox URL bar, you'll get the escaped version, so you need to unescape that. Let's use this: data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAABaklEQVQ4jc2TO0hbYQBGz42pMVDrINQuGvrCpZOSoQ4WoZoOxVk7FISCQxy6BEUtSIeCIBTsYCl0cHEQV0GU0lcIIkrpUEGNVYdi6qOtiTH+ubn3c4iTOPSaxbOfM30fkuokzUoy+n/MqVOHpDkP4lk+WpIMUM7FcCxJuqAMgK8U+dzAp5Us9QOr3gJm6RvbLRG2WyI4v3dIH7vcf7VO8/BP9jI2ZqGD3HQN6R/9zCz3AuCqwIeVIfwAf2L9BB+1ocwhtS96cBtGiPfdYnLxgMejmySivRSCtbDxlp3qML/+LbGfXSOV/l4MYNsEwo1cuXObTGcXgfEjynwWNdf8OPkDTLyVQNM0ztZ7Hoae8Tk5TM7+y5PwVDFQFXtO+vUb5LqkooNY1lUaXiaRxMTTIL7dB5hEO2U3u7lReQ+Y5O71CBX+KvC6nPmNMb2LN8sUDiVJl2MH+RJ8xwd8KSHwFRXvPCdvd86fOqETu2NuOELmXqEAAAAASUVORK5CYII=
Attachment #684797 - Flags: review?(gavin.sharp) → feedback+
Attached patch Patch (deleted) — Splinter Review
Unescaped Image data
Attachment #684797 - Attachment is obsolete: true
Attachment #685382 - Flags: review?(gavin.sharp)
Attachment #685382 - Flags: review?(gavin.sharp) → review+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
Comment on attachment 685382 [details] [diff] [review] Patch Joanne points out via email that it would be good to uplift this change to Aurora/Beta to avoid having to wait until Firefox 20. This is a trivial image swap, no risk and no string changes/etc.
Attachment #685382 - Flags: approval-mozilla-beta?
Attachment #685382 - Flags: approval-mozilla-aurora?
Attachment #685382 - Flags: approval-mozilla-beta?
Attachment #685382 - Flags: approval-mozilla-beta+
Attachment #685382 - Flags: approval-mozilla-aurora?
Attachment #685382 - Flags: approval-mozilla-aurora+
The eBay logo for non en-US builds is not updated.
(In reply to Scoobidiver from comment #15) > The eBay logo for non en-US builds is not updated. Good catch, I had forgotten about those. We'll have to fix that in a followup, I filed bug 818328.
Blocks: 580510
Guess you want this for comm-% and thunderbird, too?
"Want" is a strong word, I don't think there's a strong driver for comm-% and thunderbird to do anything (of course they're free to...).
Verified fixed on the latest beta, Firefox 18 beta 3. User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:18.0) Gecko/20100101 Firefox/18.0 Build ID: 20121205060959
Verified fixed on the latest beta, Firefox 20 beta 6. (Build ID: 20130320062118) User Agent: Mozilla/5.0 (X11; Linux i686; rv:20.0) Gecko/20100101 Firefox/20.0 User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:20.0) Gecko/20100101 Firefox/20.0 User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:20.0) Gecko/20100101 Firefox/20.0
Setting this to verified, based on comment 20.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: