Bug 1390271 - in few of ui dialogs the fields position is pushed down or cut after replacing to the new list boxes
Summary: in few of ui dialogs the fields position is pushed down or cut after replacin...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.WebAdmin
Version: 4.1.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ovirt-4.1.2
: 4.1.2
Assignee: Alexander Wels
QA Contact: Pavel Novotny
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-10-31 15:23 UTC by Sharon Gratch
Modified: 2019-04-28 13:36 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-05-23 08:18:08 UTC
oVirt Team: UX
Embargoed:
rule-engine: ovirt-4.1+
rule-engine: planning_ack+
oourfali: devel_ack+
pstehlik: testing_ack+


Attachments (Terms of Use)
screenshot (40.28 KB, application/vnd.oasis.opendocument.graphics)
2016-10-31 15:23 UTC, Sharon Gratch
no flags Details
screenshot (48.60 KB, application/vnd.oasis.opendocument.graphics)
2016-10-31 15:24 UTC, Sharon Gratch
no flags Details
screenshot (42.90 KB, application/vnd.oasis.opendocument.graphics)
2016-10-31 15:24 UTC, Sharon Gratch
no flags Details
screeenshot (53.04 KB, application/vnd.oasis.opendocument.graphics)
2016-10-31 15:25 UTC, Sharon Gratch
no flags Details
VM network interface dialog (103.05 KB, image/png)
2017-02-01 18:41 UTC, Pavel Novotny
no flags Details
Import VM dialog (82.83 KB, image/png)
2017-02-01 18:42 UTC, Pavel Novotny
no flags Details
screencast: Import VM dialog - Load button (3.44 MB, application/ogg)
2017-03-16 17:34 UTC, Pavel Novotny
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 66209 0 master MERGED webadmin: Network interface edit layout 2016-11-08 20:50:45 UTC
oVirt gerrit 66283 0 master MERGED webadmin: VM import layout 2016-11-09 14:12:08 UTC
oVirt gerrit 72309 0 master MERGED webadmin: fix layout issues 2017-02-14 21:38:24 UTC
oVirt gerrit 72315 0 ovirt-engine-4.1 MERGED webadmin: fix layout issues 2017-02-14 22:26:14 UTC
oVirt gerrit 74987 0 ovirt-engine-4.1 MERGED webadmin: import dialog load button 2017-04-02 08:04:24 UTC

Description Sharon Gratch 2016-10-31 15:23:20 UTC
Created attachment 1215850 [details]
screenshot

Description of problem:
After switching to the new list boxes, it seems that few fields are pushed down within the dialog or cut.
For examples please see the v2v dialog while choosing Source to be VMware or XEN or KVM.


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


How reproducible:
100%


Steps to Reproduce:
search for problematic dialogs.

For example:
1.open "New Network Interface" dialog (vms tab - sub tab "Network Interface" -> click New

2. or open v2v dialog (vms tab-> import->source set to vmware or zen or kvm)

Actual results:
few fields are cut or overriden


Expected results:
the fields should be positioned as before.


Additional info:
Please see attachments for examples

Comment 1 Sharon Gratch 2016-10-31 15:24:03 UTC
Created attachment 1215852 [details]
screenshot

Comment 2 Sharon Gratch 2016-10-31 15:24:31 UTC
Created attachment 1215853 [details]
screenshot

Comment 3 Sharon Gratch 2016-10-31 15:25:00 UTC
Created attachment 1215854 [details]
screeenshot

Comment 4 Oved Ourfali 2016-11-01 12:26:36 UTC
We have switched only in 4.1, not in 4.0.5.
Are you sure you're working with 4.0.5?

Comment 5 Sharon Gratch 2016-11-01 14:26:37 UTC
(In reply to Oved Ourfali from comment #4)
> We have switched only in 4.1, not in 4.0.5.
> Are you sure you're working with 4.0.5?

you are right, it was tested on 4.1 (master). My mistake, sorry.

Comment 6 Alexander Wels 2016-11-07 21:22:38 UTC
This is technically 2 bugs in one, one for the network interface layout and one for the image import layout. I will make 2 separate patches one for each issue.

Comment 7 Sandro Bonazzola 2016-12-12 13:58:53 UTC
The fix for this issue should be included in oVirt 4.1.0 beta 1 released on December 1st. If not included please move back to modified.

Comment 8 Pavel Novotny 2017-02-01 18:41:09 UTC
FailedQA in
ovirt-engine-webadmin-portal-4.1.0.3-0.1.el7.noarch
rhevm-4.1.0.3-0.1.el7.noarch

The trimmed input boxes and labels have been fixed,
however the fix affected another parts of the layout:

1) VM network interface dialog
Spacing between the "Custom MAC address" text input and "Example: 00:14..." text is now too big so it seems that the hint text belongs to the "Up" icon below it.

The "Unplugged" icon at the bottom overlaps the dialog content border.
This applies only for upcoming Firefox 52b2 ESR (51 is also affected).
In Firefox 45 ESR and Chromium 55 it is rendered fine.
See attached screenshot.

2) Import VM dialog
The "Load" button is now aligned to the left
(which is IMO actually much better than previous alignment to the right side),
but now the button fills the whole width of the dialog. The button rectangle outline has the same dimensions, but it is clickable on the whole width of the dialog.
This looks same in FF 45 ESR, 51, 52b2 ESR & Chr. 55.
See the second screenshot attached.

Comment 9 Red Hat Bugzilla Rules Engine 2017-02-01 18:41:15 UTC
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 10 Pavel Novotny 2017-02-01 18:41:45 UTC
Created attachment 1246771 [details]
VM network interface dialog

Comment 11 Pavel Novotny 2017-02-01 18:42:32 UTC
Created attachment 1246772 [details]
Import VM dialog

Comment 12 Pavel Novotny 2017-03-16 17:32:51 UTC
FailedQA in 
rhevm-4.1.1.4-0.1.el7.noarch
ovirt-engine-webadmin-portal-4.1.1.4-0.1.el7.noarch

Tested browsers: Firefox 45 ESR (currently outdated by version 52), Firefox 52 ESR, Chromium 56.

(In reply to Pavel Novotny from comment #8)
> FailedQA in
> ovirt-engine-webadmin-portal-4.1.0.3-0.1.el7.noarch
> rhevm-4.1.0.3-0.1.el7.noarch
> 
> The trimmed input boxes and labels have been fixed,
> however the fix affected another parts of the layout:
> 
> 1) VM network interface dialog
> Spacing between the "Custom MAC address" text input and "Example: 00:14..."
> text is now too big so it seems that the hint text belongs to the "Up" icon
> below it.
> 
> The "Unplugged" icon at the bottom overlaps the dialog content border.
> This applies only for upcoming Firefox 52b2 ESR (51 is also affected).
> In Firefox 45 ESR and Chromium 55 it is rendered fine.
> See attached screenshot.
> 

This is all fixed now. The New Network Interface dialog looks fine in all tested browsers.

> 2) Import VM dialog
> The "Load" button is now aligned to the left
> (which is IMO actually much better than previous alignment to the right
> side),
> but now the button fills the whole width of the dialog. The button rectangle
> outline has the same dimensions, but it is clickable on the whole width of
> the dialog.
> This looks same in FF 45 ESR, 51, 52b2 ESR & Chr. 55.
> See the second screenshot attached.

This is still present in all tested browsers.
The Load button still has a clickable area across the whole dialog width (with a small difference with the VMware source, but still).
See the attached screencast for details.

Comment 13 Red Hat Bugzilla Rules Engine 2017-03-16 17:32:57 UTC
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 14 Pavel Novotny 2017-03-16 17:34:26 UTC
Created attachment 1263781 [details]
screencast: Import VM dialog - Load button

Comment 15 Alexander Wels 2017-03-31 15:40:32 UTC
So in master this is no longer an issue due to the switch to the bootstrap buttons, but since 4.1 hasn't done that yet the gwt buttons were being temperamental in that dialog.

Comment 16 Pavel Novotny 2017-04-26 13:36:02 UTC
Verified in 
ovirt-engine-webadmin-portal-4.1.2-0.1.el7.noarch
rhevm-4.1.2-0.1.el7.noarch

Testing browsers: Firefox 52.1 ESR, Firefox 53, Chromium 57.

1) VM network interface dialog
Already fixed in previous build, no changes here.

2) Import VM dialog
The "Load" button now has correct size, it does not fill the whole dialog width as before.


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