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.

before screenshot

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.

After screenshots

Remaining tasks

Manual testing, screenshots

User interface changes

None

API changes

None

Data model changes

None

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
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.

Comments

rachel_norfolk created an issue. See original summary.

rachel_norfolk’s picture

Issue summary: View changes
rachel_norfolk’s picture

FileSize
79.93 KB
rachel_norfolk’s picture

Issue summary: View changes
rachel_norfolk’s picture

Issue summary: View changes
FileSize
64.23 KB
591 bytes
PASSED: [[SimpleTest]]: [PHP 5.5 MySQL] 113,726 pass(es). View

A patch removing the class and an after screenshot...

after screenshot

rachel_norfolk’s picture

Status: Active » Needs review
rachel_norfolk’s picture

Issue summary: View changes

Beta evaluation thing.

joelpittet’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
59.92 KB
58.76 KB

This 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 .label

joelpittet’s picture

Status: Reviewed & tested by the community » Needs work

Whoops needs work because the removal of the old CSS.

rachel_norfolk’s picture

Status: Needs work » Needs review
FileSize
1.12 KB
PASSED: [[SimpleTest]]: [PHP 5.5 MySQL] 113,944 pass(es). View
558 bytes

Sorry joelpittet, I forgot about the css.

Oh - and I REALLY love patches that do nothing more than delete code... ;-)

joelpittet’s picture

Status: Needs review » Reviewed & tested by the community

Me too:) Thank you again @rachel_norfolk

joelpittet’s picture

Issue summary: View changes
alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed ab0e9e1 and pushed to 8.0.x. Thanks!

  • alexpott committed ab0e9e1 on 8.0.x
    Issue #2573139 by rachel_norfolk, joelpittet: Followup: Remove unneeded...

Status: Fixed » Closed (fixed)

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