- Issue created by @ressa
- First commit to issue fork.
- π«π·France dydave
Thanks a lot @ressa, once again for the all the help testing and reporting in issues! π
Sorry for the late reply on this, but I've been handling other issues in the queue as well π
Thanks a lot for catching this issue, documenting it so well and for providing the solution in the merge request.
I have done a search in the code base and could not find any other occurrences for<br/>
after applying the patch.I have added a quick commit with very minor text edits:
https://git.drupalcode.org/project/admin_toolbar/-/merge_requests/155/di...
Mostly to roll-in the changes from:
#3526123-4: element hangs, when Toolbar is hidden β and #3304810-72: Toggle away Admin Toolbar completely βWould you see any more textual elements that should be adjusted/improved, before releasing a 3.6.1 patch version?
Otherwise, whenever we could have your confirmation on the latest text changes, the MR should be good to be merged π
Over to you: Needs review!
Thanks very much in advance! π
π©π°Denmark ressa CopenhagenYou're welcome @dydave, thanks for taking a look at it. And definitely no worries, I have seen all the activity in the other issues! ... and we all have a limited number of hours, for volunteering our free time to Open Source.
Great extra additions to the MR, and good call to fix those details mentioned in other issues -- it now says "Toolbar sticky" everywhere π I also think using
<br>
should be safe, but the final test will be releasing it, and see if the translation systems accepts it, though I can't see why it shouldn't ...I don't see any more textual elements that needs adjustment or to be improved on, before releasing a 3.6.1 patch version, it looks great now!
-
dydave β
committed b244ce07 on 3.x authored by
ressa β
Issue #3527462 by ressa, dydave: Fixed import of module's translated...
π«π·France dydaveSuper nice! As always!
Thanks a lot @ressa! π
Following your confirmation, I went ahead and merged the changes above at #8 π₯³
That's one more issue to be release in the upcoming 3.6.1 patch version π
Let's keep working on the other few issues so we could create a new stable release.
Thanks again very much @ressa for the great help! ππ©π°Denmark ressa CopenhagenAfter updating the Danish translations to use
<br>
a while ago, I finally got around to trying to get the latest translations, and I am happy to report back that Admin Toolbar is now 100% translated into Danish, https://localize.drupal.org/translate/projects/admin_toolbar πI caught a single lingering untranslated string in the interface, which is "milliseconds" here:
$ grep -rinoE '.{0,10}millis.{0,10}' . ./src/Form/AdminToolbarSettingsForm.php:136:ffix' => 'milliseconds',
It just needs to get wrapped in
t()
, and all is well:'#field_suffix' => $this->t('milliseconds'),
π«π·France dydaveSuper good catch @ressa! As usual! π
Indeed, it appears we missed this string π
milliseconds
https://git.drupalcode.org/project/admin_toolbar/-/blob/3.6.1/src/Form/A...Let's try to add this change to the next merge request that could get merged π
Great job on the translations! π₯³
Thanks a lot for the feedback as well on the localization work, I hope this could serve as documentation for others to find their way through the localization system on Drupal.org.
I've followed your link myself and submitted more than 20 French translations suggestions π
We've got a bit of work, but looks like we're not the worst πThanks again for all the great help keeping the module well maintained @ressa!
π©π°Denmark ressa CopenhagenThanks @dydave for a fast and positive answer as always! Great idea to include this in the merge request.
I recently worked with the deployment part of translations, and was finding that translating block titles and field names ("configuration") required too many manual steps, as opposed to interface translation, which automatically registers all
t()
-wrapped (PHP) or|trans
-ending (Twig) strings, and can become part of deploy process, all contained in a singlepo
-file.So I checked out the options, and initially planned to use https://www.drupal.org/project/config_translation_po/ β . But it still wasn't great to add another slightly working step. So I instead experimented with adding
|trans
in the relevant Twig templates.This turned out to work really well, and 99% percent of all translations are in a single .po file, one for each language. Only Site slogan, and some meta tag settings are still in language specific configuration files.
I updated the documentation pages in case you are interested:
Great that the French version got some more translations! Translating can take time, but https://www.deepl.com/ works pretty well (Tip: Sometimes it DeepL gets confused, but adding line breaks before and after tag signs (
<
and>
), and then restore line breaks before inserting in the localize.drupal.org field sometimes helps) so I also made these two 100% translated in Danish :)https://localize.drupal.org/translate/projects/token
https://localize.drupal.org/translate/projects/pathauto... and I see French is also doing well there, kudos!
Production build 0.71.5 2024