[ga-IE] Update icon and search URLs for focal.ie, now tearma.ie

RESOLVED FIXED

Status

Mozilla Localizations
ga-IE / Irish
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: flod, Assigned: flod)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

Created attachment 8645723 [details]
Focal 96x96

We have an updated icon for Fennec, but we should also update the URLs to avoid unnecessary redirects.

http://m.focal.ie/Search.aspx -> http://m.tearma.ie/Default.aspx

This one is broken
http://focal.ie/Search.aspx

Should use
http://www.tearma.ie/Search.aspx

@Kevin
Should we also update the name and use the same fixes for desktop?
Flags: needinfo?(kscanne)
http://hg.mozilla.org/releases/l10n/mozilla-aurora/ga-IE/file/fb0485403911/mobile/searchplugins/focal.xml

Will also need to know if we need to update the description for desktop, and how.

Comment 2

3 years ago
(In reply to Francesco Lodolo [:flod] from comment #0)
> Created attachment 8645723 [details]
> Focal 96x96
> 
> We have an updated icon for Fennec, but we should also update the URLs to
> avoid unnecessary redirects.
> 
> http://m.focal.ie/Search.aspx -> http://m.tearma.ie/Default.aspx
> 
> This one is broken
> http://focal.ie/Search.aspx
> 
> Should use
> http://www.tearma.ie/Search.aspx
> 
> @Kevin
> Should we also update the name and use the same fixes for desktop?

Yes, name should change from "focal.ie" to "tearma.ie" everywhere, desktop too.

> Will also need to know if we need to update the description for desktop, and how.

For the description, it suffices to substitute  s/focal/tearma/:

<Description>tearma.ie: Cuardach Comhtháite</Description>

Thanks!
Flags: needinfo?(kscanne)
Blocks: 1179109
Created attachment 8646187 [details] [diff] [review]
bug1192836.patch

* Replace focal.ie with tearma.ie
* Use .ico with 16px+32px for desktop, use 96px png for Android. Icons used on desktop are taken from the website, the Android one was approved by the provider with a light gray background.
* Use alphabetical order in list.txt

Both plugins are available for testing here (tested on Nightly, Nexus 7)
https://l10n.mozilla-community.org/~flod/testsp/
Attachment #8646187 - Flags: review?(l10n)

Comment 4

3 years ago
Comment on attachment 8646187 [details] [diff] [review]
bug1192836.patch

Review of attachment 8646187 [details] [diff] [review]:
-----------------------------------------------------------------

hah, who would have figured that I'd r- this?

I'd like to make it more prominent that we're not replacing, but that we're following a branding change.

Can you `hg mv` the xml files instead, and change the commit message to something like "focal.ie is now tearma.ie" or so?

I wish there was a perma-url for the news items on http://www.tearma.ie/NewsArchive.aspx that announce the change. That would have been a nice additionn to the commit message.
Attachment #8646187 - Flags: review?(l10n) → review-
Created attachment 8646209 [details] [diff] [review]
bug1192836v1.patch

New try with hg mv and updated commit message
Attachment #8646209 - Flags: review?(l10n)
Attachment #8646187 - Attachment is obsolete: true

Comment 6

3 years ago
Comment on attachment 8646209 [details] [diff] [review]
bug1192836v1.patch

Thanks, r=me
Attachment #8646209 - Flags: review?(l10n) → review+
You need to log in before you can comment on or make changes to this bug.