Problem/Motivation

When the content translation module was renamed from translation entity module, the filename for the admin CSS file was not renamed. The module info calls content_translation.admin.css, but the filename is still translation_entity.admin.css.

Proposed resolution

Rename translation_entity.admin.css to content_translation.admin.css.

This bug was initially reported in #2049387: Locale string change styles applied too excessively, when columns weren't properly showing up. This bug is being filed separately because the original bug reported two separate issues.

Original report by [GaborHojtsy]

Following #1986400: Table style update: Table style update, a bug appeared in Drupal core due to styling changes.

Content translation bug:

1. Enable content translation module.
2. Go to content settings under regional and language.
3. Enable translation for Articles.
4. Notice how image field columns are NOT indented, appearing like they are also just same level fields.

This is due to the 'column' class not being effective to assign an increased padding to these items.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RainbowArray’s picture

Status: Active » Needs review
FileSize
42.32 KB
42.63 KB
1.99 KB

I've attached a patch that renames the CSS file from translation_entity.admin.css to content_translation.admin.css. This should fix a few problems!

Before:
no-indent.png

After:
indent.png

wroxbox’s picture

Status: Needs review » Reviewed & tested by the community

Works just like it should.

No more 404 and layout is fixed too.

Gábor Hojtsy’s picture

Issue tags: +D8MI, +sprint, +language-content

Yay, looks good! Thanks!

Gábor Hojtsy’s picture

BTW while looking at this, I found #2060271: Content translation module admin page does not support RTL. Anybody wanna help there? :)

Gábor Hojtsy’s picture

Issue tags: +Regression

Tagging as regression.

Gábor Hojtsy’s picture

Issue tags: +Quick fix

Tagging as quick fix too.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Gábor Hojtsy’s picture

Issue tags: -sprint

Woot, thanks!

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

Anonymous’s picture

Issue summary: View changes

Trying to get issue links to show up.