Closed Bug 269825 Opened 20 years ago Closed 20 years ago

update bookmarks.html

Categories

(Mozilla Localizations :: nl / Dutch, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: annevk, Assigned: u81239)

References

Details

Attachments

(3 obsolete files)

There are several links in bookmarks.html that need to be updated before nl-NL
is going to be released.

"Laatste nieuws" should use <http://www.nos.nl/export/nosnieuws-rss.xml> or
<http://nu.nl/deeplink_rss2/index.jsp?r=Algemeen> as FEEDURL. I believe most
people prefer to first since it isn't commercial. HREF should point to
<http://www.nos.nl/nieuws/nieuws/> or <http://nu.nl/>. Suggestions are welcome.

Instead of pointing to <http://www.mozillastore.com/> we should point to
<http://store.mozilla-europe.org/nl/> using a translation like "Mozilla Winkel".
This will make us more future proof.

Instead of pointing to <http://www.mozilla.org/> we should point to
<http://www.mozilla-europe.org/nl/> since that's the official NL site for Mozilla.

Instead of using <http://www.urbandictionary.com/define.php?term=%s> I, among
others, suggest we use
<http://www.vandale.nl/opzoeken/woordenboek/?zoekwoord=%s> since that's much
more usable for us Dutch people.

We should also drop the trailing slash of <http://start.mozilla.org/firefox/>
since then the link will always work.

I CC'ed Benjamin and Axel to comment on the suggestions.
Attached patch Patch with proposed changes (obsolete) — Splinter Review
Nice ones. I created a patch for these changes, attached.
Comment on attachment 166003 [details] [diff] [review]
Patch with proposed changes

looks good to me
Attachment #166003 - Flags: approval-aviary?
Attached patch New patch (conflict) (obsolete) — Splinter Review
There was a conflict with the patch from bug 267130. Updating patch.
Attachment #166003 - Attachment is obsolete: true
Attachment #166003 - Flags: approval-aviary?
Attachment #166012 - Flags: approval-aviary?
Attachment #166012 - Flags: approval-aviary? → approval-aviary+
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
REOPENING since some additional changes are needed. Sorry about the bugspam.

The string "Raak betrokken - Help om Firefox te verspreiden!" should be replaced
with "Help mee - verspreid Firefox" to be more consistent with Mozilla Europe.

The link to Van Dale should be placed above Dictionary.com and Dictionary.com
should have (EN) at the end. The link to Mozilla Store should have (EN) as well
to be consistent with the other links.

Mozbrowser.nl should become MozBrowser.nl
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee: Martijn.Ras → lholst
Status: REOPENED → NEW
Attached patch Patch with Anne's comments (obsolete) — Splinter Review
I created a patch with Anne's comments.

This is my first patch, so I hope there will be no problems..
Attachment #166305 - Flags: approval-aviary?
Attachment #166012 - Attachment is obsolete: true
Attachment #166305 - Flags: approval-aviary?
Attachment #166305 - Flags: approval-aviary+
Blocks: 270474
No longer blocks: 267130
Comment on attachment 166305 [details] [diff] [review]
Patch with Anne's comments

committed
Attachment #166305 - Attachment is obsolete: true
Status: NEW → RESOLVED
Closed: 20 years ago20 years ago
QA Contact: T.C.Witte → Martijn.Ras
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: