- Issue created by @briangon
- First commit to issue fork.
- 🇦🇹Austria tgoeg
I added a MR that incorporates your changes (thank you very much for your contribution!) and improves on the way the label is inserted (
aria-label
), as that seems to be best practice now from what I gathered.
Additionally, the strings are also configurable now.I am unsure how these strings could be made translatable as I am no real dev at all.
But for the time being, I can at least localize these strings by entering them in the admin config (and I guess they should be configurable anyway, as all other strings are, as well).I did not want to delay merging b/c of the translation aspect. Let's get this reviewed/merged first and then probably open another issue for the translation.
If anyone wants to quickly test this in their
composer.json
, this is the magic sauce:"drupal/admin_feedback": { "Improve a11y": "https://git.drupalcode.org/project/admin_feedback/-/merge_requests/10.patch" },