i18n: Dutch #950
i18n: Dutch #950
Conversation
| 'de' => 'Deutsch', | ||
| 'en' => 'English', | ||
| 'fr' => 'Français', | ||
| 'nl' => 'Nederlands', |
There was a problem hiding this comment.
Note: we should add this line to other files + gen.lang.cz (Čeština) is missing here.
There was a problem hiding this comment.
Done Alkarex@501e331
Maybe in the future we could extract this data, which is the same for all languages.
| 'submit' => 'Opslaan', | ||
| 'truncate' => 'Verwijder alle artikelen', | ||
| ), | ||
| 'auth' => array( |
There was a problem hiding this comment.
This section does not seem to be the same structure as other languages.
There was a problem hiding this comment.
Can't find anything wrong here.
On 21-8-2015 13:39, Alexandre Alapetite wrote:
In app/i18n/nl/gen.php
#950 (comment):
'create' => 'Opslaan','disable' => 'Uitzetten','empty' => 'Leeg','enable' => 'Aanzetten','export' => 'Exporteren','filter' => 'Filteren','import' => 'Importeren','manage' => 'Beheren','mark_read' => 'Markeer als gelezen','mark_favorite' => 'Markeer als favoriet','remove' => 'Verwijder','see_website' => 'Bekijk website','submit' => 'Opslaan','truncate' => 'Verwijder alle artikelen',- ),
- 'auth' => array(
This section does not seem to be the same structure as other languages.
—
Reply to this email directly or view it on GitHub
https://github.com/FreshRSS/FreshRSS/pull/950/files#r37626640.
There was a problem hiding this comment.
Thanks for the feedback. I was referring to the auth section:
|
Yes it is Dutch (Nederlands). I'll doublecheck the file with the remark about "This section does not seem to be the same structure as other languages.". But as far as I can tell now it is original only English is translated to Dutch. My site at http://www.nieuwskop.be (version 1.1.2.beta) is working perfectly with it. By the way, I'm also the translator for the Dutch languages files of the open source CMS PHP-Fusion. Please keep in mind that I have two versions of Dutch translations available at github. Suggestion for detecting available languages: Scan app\i18n\ for available languages and match them with a predefined table for full language display. |
|
Couldn't find anything wrong in gen.php. |
|
Completely unrelated but @Wanabo, could you share your CSS rules? I love the density you achieved on your website. |
Done Alkarex@b3ae940
See in
It is also good to double-check that |
|
I've rechecked every file for version 1.1.2 beta. I updated the files when I saw non translated parts on my website when I updated from version 1.0.0. to 1.1.2 beta. That is not a good strategy! It seems I missed some additions. Now at least all files have the same number of lines and did a file comparison between 'en' and 'nl'. I've noticed a lot of language definitions return in different files. Isn't it possible to define returning strings in a master language file. That would save work. For example install.php and admin.php share a lot of content. Or an on board language editor. |
|
@Alkarex > you should consider branch 1.1.2 of @Wanabo: https://github.com/Wanabo/FreshRSS-Dutch-translation/tree/1.1.2-beta |
|
@marienfressinaud Yes, that is the one I took ( https://github.com/Wanabo/FreshRSS-Dutch-translation/blob/c2385d656f598ae8344506e67337bad4e8dd3dac/gen.php ) I can see that there has been some changes today. |
Merge changes from https://github.com/Wanabo/FreshRSS-Dutch-translation FreshRSS#949
|
I have merged today's changes from https://github.com/Wanabo/FreshRSS-Dutch-translation/tree/1.1.2-beta : Alkarex@b33004c |
|
@Alwaysin It's very simple. I added some modified css from base-theme to dark.css and left base-theme original. You can download Theme DarkDense (inspired on Theme Dark) here, http://www.nieuwskop.be/theme-upload/DarkDense.zip |
|
Thanks for the theme! Works great 👍 Maybe we could add it in FreshRSS by default? |
|
@Alkarex > is the PR ready to merge? @Wanabo and @Alwaysin > note there is an extension to add CSS rules: I think you don't need a specific theme to achieve this ;) |
|
@marienfressinaud Yes, ready. The PR integrates the latest changes from https://github.com/Wanabo/FreshRSS-Dutch-translation/commits/1.1.2-beta |
|
Yeah with xExtension-CustomCSS all themes become dense with just a few lines of CSS. I've setup a small tutorial. Achieve more dense themes with xExtension-CustomCSS |
|
Merged! Thank you @Wanabo :) |
#949
https://github.com/Wanabo/FreshRSS-Dutch-translation