this is a follow up for #1998528: Better UI for translation of configuration, the configuration Translation UI is not responsive and looks a bit strange on a Mobile Phone.
As the String Translation UI is also not responsive, this is not super bad right now. Will check with the Mobile people how to handle this.

CommentFileSizeAuthor
#7 1999156-config-translation-responsive-7.patch1.02 KBSchnitzel
#6 1999156-config-translation-responsive-6.patch1.01 KBSchnitzel
#5 edit-site-info-responsive.png63.82 KBnonsie
#4 1999156-config-translation-responsive-4.patch843 bytesSchnitzel
#2 Edit_german_translation_for_Kontoeinstellungen___local.drupal.jpg876.11 KBSchnitzel
#2 Edit_german_translation_for_Kontoeinstellungen___local.drupal.jpg1.3 MBSchnitzel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented

Woot, thanks!

Schnitzel’s picture

Schnitzel CreditAttribution: Schnitzel commented
FileSize
Edit_german_translation_for_Kontoeinstellungen___local.drupal.jpg1.3 MB
Edit_german_translation_for_Kontoeinstellungen___local.drupal.jpg876.11 KB

Here a Screenshot how it looks in Responsive:
Edit_german_translation_for_Kontoeinstellungen___local.drupal.jpg

and here desktop:
Edit_german_translation_for_Kontoeinstellungen___local.drupal.jpg

Question:
- How do we explain the difference between Source and Translation?

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented

Can you post the patch please? I'd love to get people review this :)

Schnitzel’s picture

Schnitzel CreditAttribution: Schnitzel commented
Status: Active » Needs review
FileSize
1999156-config-translation-responsive-4.patch843 bytes

here the patch for responsive, one change already: we're hiding the label on the translation in responsive

nonsie’s picture

nonsie
she/her
Primary language Estonian
CreditAttribution: nonsie commented
FileSize
edit-site-info-responsive.png63.82 KB

This is what it looks like in responsive with the label removed for the translation which IMHO improves the UI significantly.
Tested in FF, Chrome, Safari but not in Opera. Breaks in IE8 but we do not care about that.

Schnitzel’s picture

Schnitzel CreditAttribution: Schnitzel commented
FileSize
1999156-config-translation-responsive-6.patch1.01 KB

this is a new patch, which does exactly the same, just it does not use "display:none" it uses the same behavior as "element-invisible", which is better for accessibility.

Schnitzel’s picture

Schnitzel CreditAttribution: Schnitzel commented
FileSize
1999156-config-translation-responsive-7.patch1.02 KB

also fixxing http://drupal.org/node/1952394#comment-7433042 (css issues)

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Needs review » Fixed

Amazing, thanks!

Status: Fixed » Closed (fixed)

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