Bug 1804675 - [ja_JP] Text alignment correction needed on administration -> configure -> instance type -> add page
Summary: [ja_JP] Text alignment correction needed on administration -> configure -> in...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.WebAdmin
Version: 4.4.0
Hardware: All
OS: Linux
medium
medium
Target Milestone: ovirt-4.4.3
: ---
Assignee: Lucia Jelinkova
QA Contact: Ivana Saranova
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-02-19 11:48 UTC by Bhushan Barve
Modified: 2020-11-11 06:42 UTC (History)
11 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2020-11-11 06:42:06 UTC
oVirt Team: Virt
Embargoed:
pm-rhel: ovirt-4.4+
pm-rhel: ovirt-4.5?
mtessun: planning_ack+
ahadas: devel_ack+
bbarve: testing_ack+


Attachments (Terms of Use)
ja_JP_text_alignment (80.46 KB, image/png)
2020-02-19 11:48 UTC, Bhushan Barve
no flags Details
Screenshot of the Japanese locale (78.99 KB, image/png)
2020-07-07 09:35 UTC, Lucia Jelinkova
no flags Details
Screenshot of the English locale (70.38 KB, image/png)
2020-07-07 09:36 UTC, Lucia Jelinkova
no flags Details
Issue is still present, the last character is still falling on the new line (179.92 KB, image/png)
2020-10-02 20:05 UTC, Krishna Babu K
no flags Details
string comparison (45.61 KB, image/png)
2020-10-02 20:10 UTC, Krishna Babu K
no flags Details
String comparison (54.25 KB, image/png)
2020-10-02 20:14 UTC, Krishna Babu K
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 111239 0 master MERGED core: Translation pull from zanata (2020-09-11) 2021-01-11 08:55:13 UTC
oVirt gerrit 111364 0 master MERGED webadmin: Small VM popup UX changes 2021-01-11 08:55:13 UTC

Description Bhushan Barve 2020-02-19 11:48:32 UTC
Created attachment 1664014 [details]
ja_JP_text_alignment

Description of problem: The text alignment on  administration -> configure -> instance type -> add page for ja_JP locale needs to be corrected. On the resource allocation tab, there is option for 'Memory Balloon Device Enabled'. For equivalent ja_JP string, the last character is falling on the new line. Please refer attached screenshot.


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Sharon Gratch 2020-06-09 12:58:52 UTC
Bhushan, can you please attach screenshot for the instance type->add page?

Comment 2 Bhushan Barve 2020-06-10 12:02:29 UTC
Hi Sharon,

The screen-shot is available in the attachments section.

Comment 3 Lucia Jelinkova 2020-07-07 09:35:55 UTC
Created attachment 1700129 [details]
Screenshot of the Japanese locale

Comment 4 Lucia Jelinkova 2020-07-07 09:36:28 UTC
Created attachment 1700130 [details]
Screenshot of the English locale

Comment 5 Lucia Jelinkova 2020-07-07 09:56:06 UTC
I've simulated the issue but in my case the word wrapping looks a little bit different. Actually, it looks very similar to English locale (see attachments). 

Although you can see there is quite a lot of space in the dialog, fixing it is not as simple as setting widget's width. The Instance type dialog is the same as the VM edit dialog, only some widgets are hidden. Memory balloon is instance type field, it has the special instance type widget next to it in the VM dialog. To keep all the special instance type widgets aligned in the dialog, the widget's width is restricted.

I can see 2 options:
1. Accept the fact that there are many fields, locales, browsers, fonts and that not all fields would be aligned in one line
2. Increase the width of the popup dialog - 30px should be enough for the English locale, it should fix also the author's layout but not my Japanese layout (I needed to increase width by 120px to align it to one line). Note that this could break layout of other wrapped fields in the dialog.

Comment 6 Arik 2020-08-05 08:39:12 UTC
(In reply to Lucia Jelinkova from comment #5)
> I can see 2 options:
> 1. Accept the fact that there are many fields, locales, browsers, fonts and
> that not all fields would be aligned in one line
> 2. Increase the width of the popup dialog - 30px should be enough for the
> English locale, it should fix also the author's layout but not my Japanese
> layout (I needed to increase width by 120px to align it to one line). Note
> that this could break layout of other wrapped fields in the dialog.

Yeah, I wouldn't go with changing the width of the popup dialog considering the possible implications of that.

Would it make any difference for the Japanese translation if we omit the 'Device' part?

Comment 7 Bhushan Barve 2020-08-05 09:09:39 UTC
(In reply to Arik from comment #6)
> (In reply to Lucia Jelinkova from comment #5)
> > I can see 2 options:
> > 1. Accept the fact that there are many fields, locales, browsers, fonts and
> > that not all fields would be aligned in one line
> > 2. Increase the width of the popup dialog - 30px should be enough for the
> > English locale, it should fix also the author's layout but not my Japanese
> > layout (I needed to increase width by 120px to align it to one line). Note
> > that this could break layout of other wrapped fields in the dialog.
> 
> Yeah, I wouldn't go with changing the width of the popup dialog considering
> the possible implications of that.
> 
> Would it make any difference for the Japanese translation if we omit the
> 'Device' part?

May be Yuko can answer this part well regarding Japanese translation.

Comment 8 Yuko Katabami 2020-08-05 09:31:07 UTC
Thanks Bhushan for checking with me.

I think it is the best if the width of the dialog can be increased.
Otherwise, you might be able to reword it "enable memory ballooning" (instead of just removing the word "device" from the string)
The translation is "メモリーバルーニングを有効にする"
Could this fit in the space?

Comment 9 Arik 2020-08-05 11:57:02 UTC
We have similar cases like:
VirtIO-SCSI Enabled
I/O Threads Enabled
Multi Queues enabled

So I wouldn't rule out "Memory Balloon/Ballooning Enabled"
Seems like with that change the translation is more likely to fit into a single line

Comment 10 Yuko Katabami 2020-08-05 12:03:27 UTC
Please let me know if you would like to change the Japanese translation in Zanata.

Comment 11 Arik 2020-08-05 16:32:52 UTC
Lucia, what do you think about this approach?

Comment 12 Lucia Jelinkova 2020-08-13 13:21:20 UTC
I like Memory Ballooning Enabled label because it is consistent with the rest of the page. However, I'll not be able to verify since my fonts display a little bit differently than Bhushan's.

Comment 13 Arik 2020-09-21 15:10:43 UTC
(In reply to Yuko Katabami from comment #10)
> Please let me know if you would like to change the Japanese translation in
> Zanata.

Yes, let's change the Japanese translation in Zanata please

Comment 14 Yuko Katabami 2020-09-22 01:06:06 UTC
I've applied the change to Japanese translation in Zanata at: https://zanata.phx.ovirt.org/project/view/ovirt-engine
Please let me know if there is anything else I can help you.

Comment 15 Krishna Babu K 2020-10-02 20:05:50 UTC
Created attachment 1718519 [details]
Issue is still present, the last character is still falling on the new line

Tested on version 4.4.3.5-0.5, the last character is still falling on the new line.

Seems like the corrected translation (as per comment 14) is not merged in this version. As per my understanding, the UI available in this version has the string as "メモリーバルーンデバイスを有効にする" whereas the actual translation source has string as "メモリーバルーニングを有効にする".

Comment 16 Krishna Babu K 2020-10-02 20:10:28 UTC
Created attachment 1718520 [details]
string comparison

String comparison between version 4.4.3.5-0.5 UI and https://zanata.phx.ovirt.org/project/view/ovirt-engine

Comment 17 Krishna Babu K 2020-10-02 20:14:00 UTC
Created attachment 1718521 [details]
String comparison

String comparison between version 4.4.3.5-0.5 UI and https://zanata.phx.ovirt.org/project/view/ovirt-engine

Comment 18 Arik 2020-10-04 09:58:39 UTC
Sharon, who takes care to synchronize the translations from zanata to the build?

Comment 19 Sharon Gratch 2020-10-06 12:53:50 UTC
(In reply to Arik from comment #18)
> Sharon, who takes care to synchronize the translations from zanata to the
> build?

The UX team takes care of that. 
Scott, can you please sync the Japanese engine translations as part of https://gerrit.ovirt.org/111239 or do you prefer to update it on a separate patch?

Comment 20 Scott Dickerson 2020-10-07 14:41:36 UTC
(In reply to Sharon Gratch from comment #19)
> (In reply to Arik from comment #18)
> > Sharon, who takes care to synchronize the translations from zanata to the
> > build?
> 
> The UX team takes care of that. 
> Scott, can you please sync the Japanese engine translations as part of
> https://gerrit.ovirt.org/111239 or do you prefer to update it on a separate
> patch?

I will update the translation pull on patch 111239 and reference this BZ.

Comment 21 Scott Dickerson 2020-10-07 14:59:59 UTC
Only the key "memoryBalloonDeviceEnabled" in CommonApplicationConstants_ja_JP.properties has been updated.

Comment 22 Sandro Bonazzola 2020-10-07 15:39:08 UTC
Moving back to POST being https://gerrit.ovirt.org/#/c/111239/ still being reviewed and not merged yet.

Comment 23 Arik 2020-10-19 07:27:27 UTC
https://gerrit.ovirt.org/#/c/111239/ is merged

Comment 25 Ivana Saranova 2020-10-26 20:58:44 UTC
Steps:
1) Open AdminPortal with ja_JP locale
2) Go to Administration -> Configure -> Instance Types -> New -> Resource Allocation
3) Check that the Memory Ballooning Enabled translation isn't falling into a new line

Results:
The translation for Memory Ballooning Enabled is on one line.

Verified in:
ovirt-engine-webadmin-portal-4.4.3.8-0.1.el8ev.noarch
ovirt-engine-4.4.3.8-0.1.el8ev.noarch
firefox-82.0-4.fc31.x86_64

Comment 26 Sandro Bonazzola 2020-11-11 06:42:06 UTC
This bugzilla is included in oVirt 4.4.3 release, published on November 10th 2020.

Since the problem described in this bug report should be resolved in oVirt 4.4.3 release, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.