Bug 1136440 - [UX] New VM dialog: storage section in Resource Allocation is too crowded.
Summary: [UX] New VM dialog: storage section in Resource Allocation is too crowded.
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.WebAdmin
Version: ---
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
: ---
Assignee: Doron Fediuck
QA Contact: Pavel Stehlik
URL:
Whiteboard:
: 1105887 1282386 (view as bug list)
Depends On:
Blocks: 1048976
TreeView+ depends on / blocked
 
Reported: 2014-09-02 14:52 UTC by Eldan Hildesheim
Modified: 2022-07-05 13:34 UTC (History)
16 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-04-06 11:17:22 UTC
oVirt Team: SLA
Embargoed:


Attachments (Terms of Use)
vm allocation with ux and graphic problems (135.35 KB, application/octet-stream)
2014-09-02 14:52 UTC, Eldan Hildesheim
no flags Details
vm allocation fix (313.32 KB, application/octet-stream)
2014-09-02 14:53 UTC, Eldan Hildesheim
no flags Details
new vm (119.00 KB, application/octet-stream)
2014-09-15 13:21 UTC, Eldan Hildesheim
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker RHV-46983 0 None None None 2022-07-05 13:34:32 UTC
oVirt gerrit 39489 0 master MERGED webadmin: added the chain icon also to instance type field itslef Never

Description Eldan Hildesheim 2014-09-02 14:52:45 UTC
Created attachment 933815 [details]
vm allocation with ux and graphic problems

Description of problem:
-Allocation area is to crowded
-Different UX for:
  Adding disk from Grid VM > sub tab disks and
  Adding Disks from New VM Dialogue > left tab Allocation
-Lots of graphics style problems.
-The Solution Screen shot also solves bug:1048976

Solution:
1. Storage Allocation and Disks Allocation can have a separate left tab
2. Use same Disk dialogue as in the grid new disk.
3. Use Expand/Collapse section like in ‘run once’/virt
4. Use same font as other system font - CPU shares, CPU pinning...
5. Same font header as other headers - Memory allocation
6. Left grey Background, enlarge till the bottom.
7. Add lock/unlock icon to Instance Type as well -> this is very important since it comes to show that all properties are locked with instance type.
8. Align locked/unlock icons symbol

Comment 1 Eldan Hildesheim 2014-09-02 14:53:19 UTC
Created attachment 933816 [details]
vm allocation fix

Comment 2 Allon Mureinik 2014-09-03 08:09:43 UTC
Set to virt to drive as most items there seem to be virt-related.
Naturally, the storage team will collaborate on storage items as needed.

Michal - do we want to clone a separate BZ for the disks left tab?

Comment 3 Liz 2014-09-10 14:59:52 UTC
Eldan,

These suggestions look great. A few comments after looking closer:

1) Would you please make the new way that the user can get help on a field a bit stronger visually? It's hard to notice that this is an option in the current design.

2) The fields that you show in the new design for a disk should be reflected based on what the user can actually change in this case. For example, currently the user can't change the disk size since it's based on the source disk that it was cloned from. The new design should be accurate with respect to all fields to avoid any confusion in implementation.

Thanks,
Liz

Comment 4 Omer Frenkel 2014-09-11 06:50:44 UTC
since fields is this sub tab are SLA and storage, moving to SLA to handle this one

Comment 5 Eldan Hildesheim 2014-09-15 13:20:02 UTC
As I understood from storage developers, according to the selected above, the fields are going to be enable/disabled or not relevant.
I'm adding just a disabled "disk space" field to emphasise that the structure should be like in "New Disk" but areas in the form can be disabled.

Regarding dotted underline to show help - made the changes.
In case u need a css example:

<div style="border-bottom: 2px dotted #000; display: inline;" title="I love Deep Purple">I <3 #36013f.</div>

Comment 6 Eldan Hildesheim 2014-09-15 13:21:44 UTC
Created attachment 937619 [details]
new vm

Comment 7 Amit Aviram 2015-03-03 08:31:21 UTC
*** Bug 1105887 has been marked as a duplicate of this bug. ***

Comment 8 Tomas Jelinek 2015-04-02 11:10:53 UTC
the 39489 fixes the 7th point from the Comment 1

Comment 9 Sandro Bonazzola 2015-10-26 12:41:17 UTC
this is an automated message. oVirt 3.6.0 RC3 has been released and GA is targeted to next week, Nov 4th 2015.
Please review this bug and if not a blocker, please postpone to a later release.
All bugs not postponed on GA release will be automatically re-targeted to

- 3.6.1 if severity >= high
- 4.0 if severity < high

Comment 10 Tomer Saban 2015-11-16 09:57:29 UTC
Eldan,

Can you explain exactly what need to be changed in (4) "Use same font as other system font - CPU shares, CPU pinning..."

To which system font you mean?

Comment 11 Yaniv Kaul 2015-11-17 11:15:00 UTC
Doron - is it going into 3.6.0?

Comment 12 Doron Fediuck 2015-11-17 15:44:43 UTC
*** Bug 1282386 has been marked as a duplicate of this bug. ***

Comment 13 Doron Fediuck 2015-11-17 15:46:25 UTC
(In reply to Yaniv Kaul from comment #11)
> Doron - is it going into 3.6.0?

No. Target updated (and this should be evaluated if 4.0 has significant UI changes).

Comment 14 Red Hat Bugzilla Rules Engine 2015-11-30 20:46:26 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 15 Yaniv Kaul 2016-02-04 17:35:26 UTC
I'd argue that the order of the allocation should be different. Everyone will do something with the storage allocation - the rest is really optional (CPU - many will not touch).
Y.

Comment 16 Yaniv Kaul 2016-03-10 13:46:07 UTC
Doron - please triage or CLOSE-WONTFIX.

Comment 17 Doron Fediuck 2016-04-06 11:17:22 UTC
Closing old issues based on capacity.
Please re-open if relevant and needed.


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