Bug 1183462 - [ja_JP] [es_ES] [User Portal] String broken in 'New or Edit Network Interface' dialog.
Summary: [ja_JP] [es_ES] [User Portal] String broken in 'New or Edit Network Interface...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine
Version: 3.5.0
Hardware: x86_64
OS: Linux
low
medium
Target Milestone: ovirt-3.6.2
: 3.6.2
Assignee: Marcin Mirecki
QA Contact: Yuko Katabami
URL:
Whiteboard:
: 1183458 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-01-19 07:13 UTC by Lijun Li
Modified: 2016-04-20 01:37 UTC (History)
17 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-04-20 01:37:13 UTC
oVirt Team: Network
Target Upstream Version:
Embargoed:
ylavi: Triaged+


Attachments (Terms of Use)
[ja_JP][es_ES] [User Portal] String broken in 'New or Edit Network Interface' dialog (46.10 KB, image/png)
2015-01-19 07:15 UTC, Lijun Li
no flags Details
'New Network Interface' dialog - improvement suggestion (80.65 KB, image/png)
2015-01-23 00:41 UTC, Einav Cohen
no flags Details
same width icon (52.86 KB, image/png)
2015-01-27 10:13 UTC, Eldan Hildesheim
no flags Details
screenshots (65.35 KB, application/x-gzip)
2015-11-29 08:09 UTC, Michael Burman
no flags Details
ja-JP fixed (32.06 KB, image/png)
2016-02-28 21:55 UTC, Yuko Katabami
no flags Details
es-ES fixed (24.75 KB, image/png)
2016-02-28 21:55 UTC, Yuko Katabami
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 47032 0 master MERGED webadmin: String broken in 'New or Edit Network Interface' dialog Never
oVirt gerrit 48014 0 ovirt-engine-3.6 MERGED webadmin: String broken in 'New or Edit Network Interface' dialog Never
oVirt gerrit 48562 0 ovirt-engine-3.6 MERGED webadmin: adjusting layout of vm nic icons Never
oVirt gerrit 48638 0 None MERGED webadmin: adjusting layout of vm nic icons Never
oVirt gerrit 51146 0 master MERGED webadmin: enlarging vm nic edit panel to fit content in all browsers 2016-01-03 07:47:35 UTC
oVirt gerrit 51316 0 ovirt-engine-3.6 MERGED webadmin: enlarging vm nic edit panel to fit content in all browsers 2016-01-05 08:46:26 UTC
oVirt gerrit 51317 0 ovirt-engine-3.6.2 MERGED webadmin: enlarging vm nic edit panel to fit content in all browsers 2016-01-05 08:47:06 UTC

Description Lijun Li 2015-01-19 07:13:15 UTC
Description of problem:
[ja_JP] [es_ES] [User Portal] String broken in 'New or Edit Network Interface' dialog.

Version-Release number of selected component (if applicable):
rhevm-3.5.0-0.29.el6ev.noarch

How reproducible:
100%

Steps to Reproduce:
1. Login user portal.
2. Click Extended->Templates->Network Interfaces.
3. Click New or Edit.

Actual results:
[ja_JP] [es_ES] [User Portal] String broken in 'New or Edit Network Interface' dialog.

Expected results:
It should be displayed in one line.

Additional info:
Please see the attached screen-shot.

Comment 1 Lijun Li 2015-01-19 07:15:29 UTC
Created attachment 981343 [details]
[ja_JP][es_ES] [User Portal] String broken in 'New or Edit Network Interface' dialog

Comment 2 Einav Cohen 2015-01-22 17:08:44 UTC
*** Bug 1183458 has been marked as a duplicate of this bug. ***

Comment 3 Einav Cohen 2015-01-23 00:41:19 UTC
The Component of this BZ is web-admin, however the steps to reproduce mention that the dialog was opened from the user-portal - this doesn't matter much, since AFAIK it's the exact same dialog that is opened in the web-admin as well as the user portal (hence, closed duplicate bug 1183458).

Comment 4 Einav Cohen 2015-01-23 00:41:56 UTC
Created attachment 983109 [details]
'New Network Interface' dialog - improvement suggestion

Comment 5 Einav Cohen 2015-01-23 00:50:33 UTC
personally, I don't think that it is looking that bad (marking as Improvement); leaving the decision whether to fix this issue or not to the 'network' team. 

there are a couple of option for fixing this issue:

- allocating more real-estate to the field-labels (either at the expense of the field-inputs or along with extending the width of the dialog). 

- [my personal preference] better-align the entire dialog to a standard horizontal layout (I elaborated a little bit about it in bug 1166055, comment #1) as depicted in attachment 983109 [details]; this way, the layout is a simple two-column layout of field-label and field-input, and it is as localization-friendly as possible in a horizontal-form-layout dialog such as ours. 

'needinfo'ing Eldan as well for his opinion (cannot recall whether he already had an improvement suggestion for this dialog or not).

Comment 6 Eldan Hildesheim 2015-01-27 10:12:23 UTC
Einav's solution works the best for multi lingual text.
I will change one of the icons so it will look align with the other (attached is a rough example).
If it's ok with u guys - I will remake the icon accordingly.

Comment 7 Eldan Hildesheim 2015-01-27 10:13:16 UTC
Created attachment 984584 [details]
same width icon

Comment 9 Sandro Bonazzola 2015-11-24 15:56:12 UTC
All referenced patches are merged, shouldn't this be on modified?

Comment 10 Marcin Mirecki 2015-11-24 18:24:14 UTC
(In reply to Sandro Bonazzola from comment #9)
> All referenced patches are merged, shouldn't this be on modified?

Yes!

Comment 11 Michael Burman 2015-11-29 08:06:33 UTC
Hi,

I think this bug should be failedQa or a new one should be opened, because on chrome the new design/the fix for 'New or Edit Network Interface' dialog is completely broken and doesn't look good.

On firefox the new design looks OK.

Attaching screenshots.

Comment 12 Michael Burman 2015-11-29 08:09:15 UTC
Created attachment 1100104 [details]
screenshots

Comment 13 Einav Cohen 2015-11-30 14:04:04 UTC
(In reply to Michael Burman from comment #11)
> Hi,
> 
> I think this bug should be failedQa or a new one should be opened, because
> on chrome the new design/the fix for 'New or Edit Network Interface' dialog
> is completely broken and doesn't look good.
> 
> On firefox the new design looks OK.
> 
> Attaching screenshots.

I agree, since we want the application to look reasonable on Chrome for 3.6. 
Network team should decide how to proceed.

Comment 14 Michael Burman 2015-12-07 08:15:15 UTC
Hi Dan, 

Please reply to the need info in order to decide how to continue here, thank you)
See comment 11 ^^

Comment 15 Dan Kenigsberg 2015-12-09 12:54:43 UTC
On Chrome, field names are cut.

On both browsers, the custom mac address slips off the right hand side of the dialog, and a horizontal scroll bar shows at the bottom.

These issues should be fixed.

Comment 16 Marcin Mirecki 2015-12-09 13:48:02 UTC
I checked this only on firefox.
Needs to be redone.

Comment 17 Marcin Mirecki 2016-01-04 16:21:47 UTC
Can you please also check: https://bugzilla.redhat.com/show_bug.cgi?id=1291146
It should be fixed by this patch

Comment 19 Yuko Katabami 2016-02-28 21:55:07 UTC
Created attachment 1131346 [details]
ja-JP fixed

Comment 20 Yuko Katabami 2016-02-28 21:55:40 UTC
Created attachment 1131347 [details]
es-ES fixed

Comment 21 Yuko Katabami 2016-02-28 21:56:54 UTC
Tested on 3.6.2.6-0.1.el6 and confirmed that the text is now shown in single line in Chrome for both ja-JP and es-ES.
Moving the status to VERIFEID>


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