It's a minor thing, but visible to users.
In CKEditor context menu "Refresh" and "Copy" commands are not translatable. Also there's no translation for "Add a caption".
After short investigation it occured that scald\modules\library\dnd\plugins\ckeditor\lang\en.js script is incomplete. There should be lines:
atom_refresh: Drupal.t('Refresh'),
atom_copy: Drupal.t('Copy'),
properties_has_caption: Drupal.t('Add a caption'),
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | using-the-wrong-translation-in-drupal-for-dndck4-2529956-6.patch | 863 bytes | nagy.balint |
Comments
Comment #1
gpietrzakComment #2
nagy.balint commented@vindicator:
Those strings are only needed in the dndck4 plugin, and if you check the language file for the dndck4 plugin under modules/library/dnd/plugins/dndck4/lang/en.js
then you can see that the strings are there.
So in this case your issue is due something else. Maybe that language file is not properly included?
Comment #3
nagy.balint commentedAye seems like the issue is in function dnd_library()
Where only the legacy plugin's language file is referenced...
In fact it should load the one related to the enabled plugin.
Comment #4
nagy.balint commentedSo something like this could work.
Please test.
Comment #5
DeFr commentedI'd have written the conditional assignment, given that there's only two type of plugin right now, as
More importantly though, the problem occurred on one of our in development site too, and I can confirm that the patch fix it :-) Leaving at NR for now to let the OP chime in, but for me that's good to go, with or without the style fix.
Comment #6
nagy.balint commentedThanks,
here is the updated patch.
Comment #8
nagy.balint commentedTested it again, it seems to work fine for me. Committed.
Of course after applying this patch its not enough to clear the cache, but also a page has to be visited that has DND on it, so the new strings are added, and then they will appear under interface translation.
Comment #9
gpietrzak#6 works great for me, thank you all for the response!