Bug 981982 - New Host - External provider
Summary: New Host - External provider
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine-webadmin-portal
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 3.3.0
Assignee: Oved Ourfali
QA Contact: Artyom
URL:
Whiteboard: infra
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-07 13:52 UTC by Eldan Hildesheim
Modified: 2016-02-10 19:27 UTC (History)
7 users (show)

Fixed In Version: is8
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
oVirt Team: Infra
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Checked External Providers (81.77 KB, image/png)
2013-07-07 13:52 UTC, Eldan Hildesheim
no flags Details
Current add host dialog (25.96 KB, image/png)
2013-07-17 11:09 UTC, Oved Ourfali
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 17243 0 None None None Never

Description Eldan Hildesheim 2013-07-07 13:52:22 UTC
Created attachment 769988 [details]
Checked External Providers

Description of problem:
UI wise, page should be implemented as in the enclosed attachment.
The current proposition has 2 states:
-Unchecked External Providers (hide all related Ex. provider gui)
-Checked External Providers (show all related Ex. provider gui)


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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Oved Ourfali 2013-07-17 11:06:49 UTC
Why was it moved to the beginning of the dialog?
IMO, the DC and Cluster should remain where they are.

Also, the fact that we today have a label, an info icon, and a search button (all three aren't in your gui mockup), makes it clear for the user... having a text box with some text in it (which I didn't see we use anywhere in the current UI), is very confusing, imo.

I agree we can hide some related widgets when it isn't checked, but why changing the entire view?

Comment 2 Oved Ourfali 2013-07-17 11:09:16 UTC
Created attachment 774731 [details]
Current add host dialog

Current add host dialog

Comment 3 Oved Ourfali 2013-07-17 11:10:47 UTC
Added to current view to ease the review process of other people as well.

Comment 4 Oved Ourfali 2013-07-22 09:41:04 UTC
After discussion with Eldan we agreed that the dialog will stay similar to the current one, with the following changes:
1. s/Show External Providers/Use External Providers
2. Hide all fields in case the checkbox is not checked
3. Use the same panel as the DC/Cluster
4. Choose the first host provider automatically when enabling.

Comment 5 Artyom 2013-08-12 13:38:44 UTC
Verified on is9

Comment 6 Itamar Heim 2014-01-21 22:26:18 UTC
Closing - RHEV 3.3 Released

Comment 7 Itamar Heim 2014-01-21 22:29:43 UTC
Closing - RHEV 3.3 Released


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