Closed Bug 316525 Opened 19 years ago Closed 19 years ago

Trademark Review for Slovene (sl) Firefox 1.5

Categories

(Mozilla Localizations :: Other, defect)

defect
Not set
blocker

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Pike, Assigned: aleksr_si)

References

Details

Attachments

(1 file, 1 obsolete file)

Sorry for the late filing, I thought I had this in my internal tracking :-(. Please make the following fixes for the searchplugins: Update the yahoo plugin to the current en-US one. najdi and sigov should use najdi-si and sigov-gov-si names so that they're more likely globally unique. Update should be from https://addons.mozilla.org/searchplugins/updates/<searchplugin>.src, and don't forget to adjust list.txt. In the bookmarks, please fix the startpage to point to http://sl.start.mozilla.com/firefox (don't forget the .com). Remove the getfirefox.com alltogether, make the mozilla.org a link to mozilla.com, the store should link to http://store.mozilla-europe.org/sl and the quicksearch folder should be the second, between the personal toolbar and the Mozilla links. As a nit, we'd appreciate if you'd put the community link in the mozilla folder beneath the mozilla.com link, this is where the other teams put it, too. Thanks and sorry for letting this slip through the fingers. Please attach a patch and request approval-l10n.
Attached patch Patch with the required changes (obsolete) — Splinter Review
Thanks for the trade mark review, Axel. This is a patch against the tree with the necessary changes; it unfortunately doesn't seem to include the changes to new search plugins because their filenames have changed. I'm not sure how to go about showing those - just make a diff between the old and new versions and attach it separately perhaps?
Attachment #203111 - Flags: approval-l10n?
Comment on attachment 203111 [details] [diff] [review] Patch with the required changes just do a cvs -z3 diff -uN the 'N' will cover the file additions and removals, at least the ascii ones and I don't mind the binary pictures that much. a- this one awaiting the next patch, but this looks like the right thing is going to happen. (Just be sure that you changed the update urls in the renamed plugins, too)
Attachment #203111 - Flags: approval-l10n? → approval-l10n-
Attached patch take 2Splinter Review
Thanks for the advice. :)
Attachment #203111 - Attachment is obsolete: true
Attachment #203112 - Flags: approval-l10n?
Comment on attachment 203112 [details] [diff] [review] take 2 thanks, please close this bug when the changes are landed, we'll VERIFY if appropriate
Attachment #203112 - Flags: approval-l10n? → approval-l10n+
Changes checked in.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: