Bug 1133572 - improve layout of Add Event Notification dialog
Summary: improve layout of Add Event Notification dialog
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-webadmin
Version: 3.5
Hardware: x86_64
OS: Linux
unspecified
low
Target Milestone: ---
: 3.5.0
Assignee: Mooli Tayer
QA Contact: Martin Pavlik
URL:
Whiteboard: infra
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-08-25 13:24 UTC by Martin Pavlik
Modified: 2016-02-10 19:34 UTC (History)
8 users (show)

Fixed In Version: ovirt-3.5.0_rc2
Clone Of:
Environment:
Last Closed: 2014-10-17 12:35:27 UTC
oVirt Team: Infra
Embargoed:


Attachments (Terms of Use)
screenshot1 (41.98 KB, image/png)
2014-08-25 13:24 UTC, Martin Pavlik
no flags Details
after change 32266 (198.56 KB, application/octet-stream)
2014-09-01 16:17 UTC, Mooli Tayer
no flags Details
after after http://gerrit.ovirt.org/#/c/32266/2 (155.85 KB, application/octet-stream)
2014-09-04 08:55 UTC, Mooli Tayer
no flags Details
text (53.70 KB, application/octet-stream)
2014-09-12 07:10 UTC, Martin Pavlik
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 32266 0 master MERGED webadmin: style fixes in manage events popup. Never
oVirt gerrit 32675 0 ovirt-engine-3.5 MERGED webadmin: style fixes in manage events popup. Never

Description Martin Pavlik 2014-08-25 13:24:01 UTC
Created attachment 930482 [details]
screenshot1

Description of problem:
see screenshot

1)
on top of the dialog there is a unused white space, it could be used to extend scrolling area which is bellow

2) texts are too close to checkboxes which does not look good and when checkbox is checked it overcovers the text

Version-Release number of selected component (if applicable):
oVirt Engine Version: 3.5.0-0.0.master.20140821064931.gitb794d66.el6 

How reproducible:
100%

Steps to Reproduce:
1.users -> admin -> event notifier -> Manage Events

Comment 1 Mooli Tayer 2014-09-01 16:17:06 UTC
Created attachment 933498 [details]
after change 32266

Comment 2 Mooli Tayer 2014-09-04 08:55:06 UTC
Created attachment 934330 [details]
after after http://gerrit.ovirt.org/#/c/32266/2

Comment 3 Mooli Tayer 2014-09-09 08:57:25 UTC
Note patch also fixes

3)
Checkboxes not vertically aligned with expand arrows

Comment 4 Martin Pavlik 2014-09-12 07:09:51 UTC
would it be possible to align text in a way that second line of text begins on same horizontal position as the first one? (see screenshot text)

Comment 5 Martin Pavlik 2014-09-12 07:10:13 UTC
Created attachment 936767 [details]
text

Comment 6 Sandro Bonazzola 2014-10-17 12:35:27 UTC
oVirt 3.5 has been released and should include the fix for this issue.


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