Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
Enable the config_translation, language, locale and content_translation modules. Visit admin/config/regional/language and add a few languages (of your choice). Visit admin/reports/languages and see a list of the installed languages and any outstanding translation required. If they are all up to date, to create some outstanding translations, install a contrib module, like sparql.
Note that the name of each language is badly formatted and no displayed in the correct location in the td.
Proposed resolution
To simply remove the class="label" from the name of the language. This will mean the text appears in the correct location in the table and flow correctly within the td.
Remaining tasks
Manual testing, screenshots
User interface changes
None
API changes
None
Data model changes
None
Beta phase evaluation
| Issue category | Task, nothing is broken. |
|---|---|
| Issue priority | Normal, just a small bit of follow-up. |
| Unfrozen changes | Unfrozen because it only changes markup in the locale report around the name of the language. |
| Disruption | Not disruptive at all. |
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | interdiff-2573139-5-10.txt | 558 bytes | rachel_norfolk |
| #10 | remove-label-class-2573139-10.patch | 1.12 KB | rachel_norfolk |
| #8 | Available_translation_updates___Site-Install.png | 58.76 KB | joelpittet |
| #8 | after.png | 59.92 KB | joelpittet |
| #5 | 2573139-after-5.png | 64.23 KB | rachel_norfolk |
Comments
Comment #2
rachel_norfolkComment #3
rachel_norfolkComment #4
rachel_norfolkComment #5
rachel_norfolkA patch removing the class and an after screenshot...
Comment #6
rachel_norfolkComment #7
rachel_norfolkBeta evaluation thing.
Comment #8
joelpittetThis looks much better thank you @rachel_norfolk
One thing I think we should also remove the old CSS because nothing else is likely to use it.
#locale-translation-status-form .labelComment #9
joelpittetWhoops needs work because the removal of the old CSS.
Comment #10
rachel_norfolkSorry joelpittet, I forgot about the css.
Oh - and I REALLY love patches that do nothing more than delete code... ;-)
Comment #11
joelpittetMe too:) Thank you again @rachel_norfolk
Comment #12
joelpittetComment #13
alexpottCommitted ab0e9e1 and pushed to 8.0.x. Thanks!