The norwegian translation for tinymce use "no", not "nb"

Line 98 in wysiwyg.admin.inc
Add "no" to array
'#options' => drupal_map_assoc(array('ar', 'ca', 'cs', 'cy', 'da', 'de', 'el', 'en', 'es', 'fa', 'fi', 'fr', 'fr_ca', 'he', 'hu', 'is', 'it', 'ja', 'ko', 'nb', 'no', 'nl', 'nn', 'pl', 'pt', 'pt_br', 'ru', 'ru_KOI8-R', 'ru_UTF-8', 'si', 'sk', 'sv', 'th', 'zh_cn', 'zh_tw', 'zh_tw_utf8')),

CommentFileSizeAuthor
#1 wysiwyg-norwegian-code.patch1.29 KBtwod

Comments

twod’s picture

twod
Primary language Swedish
Location Sweden
commented
Status: Active » Needs review
StatusFileSize
new wysiwyg-norwegian-code.patch1.29 KB

According to http://en.wikipedia.org/wiki/List_of_ISO_639-1_codes nb (Norwegian Bokmål) is covered by the meta code no, so maybe we could use just 'no'? I don't know if any editor depends on it being nb tho, or even which code is more frequently used, so here's a simple patch to just add 'no' to the list.

EDIT: Bleh, just noticed the list is sorted alphabetically. Not going to upload a new patch just to fix that tho as it's Sunday and I'm feeling lazy. ;P

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

TinyMCE seems to have language packs for no and nn: http://tinymce.moxiecode.com/download_i18n.php

CKEditor ships with language files for no and nb.

torbs’s picture

torbs commented

Norwegian has to written languages :-) "bokmål" (nb) and "nynorsk" (nn). Bokmål (nb) is by far the most used and FCKeditor doesn't have a Nynorsk-translation.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

We should remove all and keep "no" only. I.e. follow the ISO standard. It's up to the editor libraries to follow the ISO standard, too. (yes, there was a reason for ISO to standardize)

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Fixed

Thanks for reporting, reviewing, and testing! Committed to all branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.