ℹ️Zur Lösung des Problems haben wir das Tag u komplett aus der Überstzung entfernt. Damit gibt es beim Import der Übersetzungen keine Fehlermeldungen mehr.

German Version (English see below)

Problem/Motivation

Leaflet enthält aktuell Zeichenfolgen, die in das < u > Tag eingebettet sind.

https://localize.drupal.org/translate/languages/de/translate?project=lea...

Offenbar gibt es einige Entwickler in der Drupal Community, die finden, dass wir das U Tag innerhalb der Übersetzungen gar nicht zulassen sollten. Die einen , weil Menschen unterstrichenen Text mit einem Link verwechseln könnten und die andern weil das < u > Tag, das eigentlich zum Unterstreichen von Text gedacht ist. in HTNL 5 nur noch dafür verwendet werden soll Anmerkungen hervorzuheben. Zum Beispiel dann, wenn Texte korrekturgelesen können Rechtschreibfehler mit dem tag < u > hervorgehoben werden. Zum Unterstreichen soll es aber nicht mehr verwendet werden.

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/u

Das letzte Wort ist hier noch nicht gesprochen und ich habe versucht alle Übersetzungen im Modul Leaflet, in denen das genannte Tag vorkommt wieder zu entfernen. Bei zwei Übersetzungen ist mir das jedoch nicht gelungen, weil diese Platzhalter enthalten. Die betreffenden Strings sind:

https://localize.drupal.org/translate/languages/de/translate?sid=2752977
https://localize.drupal.org/translate/languages/de/translate?sid=3013886

Jetzt müssen wir jemand finden, der diese Übersetzungen manuell vom Server entfernen kann. Andernfalls wird es beim Aktivieren von Leaflet und beim anschließenden Import der Übersetzungen des Moduls weiter zu einer Fehlermeldung kommen, da sich zwei der aktuell 16 von diesem Problem betroffenen Zeichenfolgen nicht über die UI des Übersetzungsservers unter Localize.drupal.org entfernen lassen, da Sie Platzhalter enthalten.

Steps to reproduce

Melde dich als Maintainer oder Translation Community Moderator an localize.drupal.org an
Trete der Gruppe der deutschen Übersetzung bei
Rufe das Projekt Leaflet auf
Versuche eine der Beiden oben angegebenen Übersetzungen über die UI zu entfernen
Du wirst eine Fehlermeldung wie die im Bild unten gezeigte erhalten.

Proposed resolution

Es muss ein Patch bereitgestellt werden, der einerseits dafür sorgt, dass Zeichenfolgen von Personen mit entsprechender Berechtigung auch dann gelöscht werden können, wenn Sie platzhalter enthalten und die betreffenden PO Dateien des jeweiligen Projektes im Hintergrund neu zusammengestellt werden, damit fehlerhafte zugelassene Übersezungen, die Platzhalter enthalten zukünftig über die UI des Übersetzungsservers gelöscht werden können.
Nicht löschbare Zeichenfolgen mit Platzhalter

Ich weiß, dass ist ein Edge-Case, der kommt aber leider ölfter vor als man denkt und deswegen sollten wir das dringend implementieren, wenn wir den Übersetzungsserver sowiso aktualisieren.

Remaining tasks

write a patch

User interface changes

API changes

Data model changes

Problem/Motivation

Leaflet currently contains strings embedded in the < u > tag.

https://localize.drupal.org/translate/languages/de/translate?project=lea...

Apparently, some developers in the Drupal community think we should not allow the < u > tag within translations. Some because people might mistake underlined text for a link and others because the < u > tag, which is intended for underlining text, should only be used in HTNL 5 to emphasize notes. For example, when proofreading texts, spelling mistakes can be emphasized with the < u > tag. However, it should no longer be used for underlining.

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/u

The last word has not yet been spoken here and I have tried to remove all translations in the Leaflet module in which the tag mentioned occurs. However, I did not succeed in doing this for two translations because they contain placeholders. The strings in question are:

https://localize.drupal.org/translate/languages/de/translate?sid=2752977
https://localize.drupal.org/translate/languages/de/translate?sid=3013886

Now we need to find someone who can manually remove these translations from the server. Otherwise, there will continue to be an error message when activating Leaflet and then importing the module's translations, because two of the currently 16 strings affected by this problem cannot be removed via the UI of the translation server at Localize.drupal.org because they contain placeholders.

Steps to reproduce

Log in as a maintainer or translation community moderator at localize.drupal.org.
Join the German translation group.
Go to the Leaflet project.
Try to remove one of the two translations given above via the UI.
You will receive an error message as shown in the image below.

Proposed resolution

A patch must be provided that ensures that strings can be deleted by persons with the appropriate authorization even if they contain placeholders and that the relevant PO files of the respective project are recompiled in the background so that incorrect approved translations that contain placeholders can be deleted via the translation server UI in the future.

Non-deletable strings containing  placeholders

I know this is an edge case, but unfortunately, it happens more often than you would think and that's why we should implement this urgently when we update the translation server anyway.

Remaining tasks

write a patch

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
07-01-2025_14-05-57.png261.17 KBjoachim namyslo

Comments

joachim namyslo created an issue. See original summary.

joachim namyslo’s picture

Issue summary: View changes
joachim namyslo’s picture

Issue summary: View changes
joachim namyslo’s picture

Issue summary: View changes
joachim namyslo’s picture

Title: Leaflet » Leaflet - Aprooved strings contianiing placeholders are undeletable
joachim namyslo’s picture

Title: Leaflet - Aprooved strings contianiing placeholders are undeletable » Leaflet - Aprooved strings contianing placeholders are undeletable
joachim namyslo’s picture

Ich habe die u - Tags in den Übersetzungen kurzerhand durch Alternativen wie < em > ersetzt. Mal sehen, ob wir jetzt noch Fehler ausgeworfen bekommen. Ich denke nicht, dass es jetzt noch Fehler beim Import gibt, wenn doch müssen wir halt noch mal drüber.

I replaced the u - tags in the translations with alternatives such as . Let's see if we can still get errors to come up. I don't think there will be any more errors during the import, but if there are, we'll just have to go over it again.

joachim namyslo’s picture

Status: Active » Needs review
joachim namyslo’s picture

Title: Leaflet - Aprooved strings contianing placeholders are undeletable » Leaflet
joachim namyslo’s picture

Issue summary: View changes

joachim namyslo’s picture

joachim namyslo’s picture

joachim namyslo’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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