Bug 1138131

Summary: [RFE][ImportDomain] The login button, when picking the targets for importing iSCSI Storage domain should be more noticeable in the GUI
Product: Red Hat Enterprise Virtualization Manager Reporter: Maor <mlipchuk>
Component: ovirt-engineAssignee: Amit Aviram <aaviram>
Status: CLOSED ERRATA QA Contact: Kevin Alon Goldblatt <kgoldbla>
Severity: low Docs Contact:
Priority: low    
Version: 3.5.0CC: acanan, amureini, gklein, iheim, lpeer, mlipchuk, rbalakri, Rhev-m-bugs, shavivi, tnisan, yeylon, ykaul, ylavi
Target Milestone: ovirt-3.6.0-rcKeywords: FutureFeature, UserExperience
Target Release: 3.6.0Flags: sherold: Triaged+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-03-09 20:39:51 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Storage RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
solution mockup
none
screenshot none

Description Maor 2014-09-04 06:46:52 UTC
Description of problem:
The login button, when picking the targets for importing iSCSI Storage domain is currently at the top of the dialog, and when picking the targets and scroll down, the user might be confused how to login

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


How reproducible:
100%

Steps to Reproduce:
1. Try to add iSCSI domain, and pick the targets
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Allon Mureinik 2014-09-04 10:53:29 UTC
Maor, please attach a screenshot explaining the problem so the UX experts can suggest an improvement.

Comment 3 Einav Cohen 2014-09-10 18:13:32 UTC
indeed, the "Login" button can be easily missed; I am wondering why the login GUI in the Import Domain dialog  (select rows, hit "login" on top-left) is so different than the login GUI within the New Domain dialog (an inline "login" button, in addition to a "login all" button on top-right)? because I think that just copying the login GUI from the "New Domain" dialog would fit here. 

Allon/Maor/Eldan: thoughts?

Comment 4 Allon Mureinik 2014-09-11 13:23:50 UTC
(In reply to Einav Cohen from comment #3)
> indeed, the "Login" button can be easily missed; I am wondering why the
> login GUI in the Import Domain dialog  (select rows, hit "login" on
> top-left) is so different than the login GUI within the New Domain dialog
> (an inline "login" button, in addition to a "login all" button on
> top-right)? because I think that just copying the login GUI from the "New
> Domain" dialog would fit here. 
> 
> Allon/Maor/Eldan: thoughts?

+1 on that.
Maor - is it an intentionally different button?

Comment 5 Eldan Hildesheim 2014-09-15 12:17:52 UTC
Created attachment 937580 [details]
solution mockup

Take the Discover Target and Login button outside the scroll.
When scrolling threw the targets the button will be static and visible all the way.

Comment 6 Maor 2014-09-30 09:43:52 UTC
(In reply to Allon Mureinik from comment #4)
> (In reply to Einav Cohen from comment #3)
> > indeed, the "Login" button can be easily missed; I am wondering why the
> > login GUI in the Import Domain dialog  (select rows, hit "login" on
> > top-left) is so different than the login GUI within the New Domain dialog
> > (an inline "login" button, in addition to a "login all" button on
> > top-right)? because I think that just copying the login GUI from the "New
> > Domain" dialog would fit here. 
> > 
> > Allon/Maor/Eldan: thoughts?
> 
> +1 on that.
> Maor - is it an intentionally different button?

One button is for login to the targets (which should be picked first)
The other button is for importing the Storage Domains discovered from the targets.
I think it is better they should be separated and different since they reflect different scenarios.
The mockup which Eldan attached seems to do the trick

Comment 7 Max Kovgan 2015-06-28 14:12:58 UTC
ovirt-3.6.0-3 release

Comment 8 Kevin Alon Goldblatt 2015-07-08 15:15:21 UTC
Verified using Version:
--------------------------------------
vdsm-4.17.0-1054.git562e711.el7.noarchrpm -qa 
ovirt-engine-3.6.0-0.0.master.20150627185750.git6f063c1.el6.noarch


Verified with the following scenario:
--------------------------------------
Steps to reproduce:
1. Via Storage Tab -> Import Domain -> Discover Targets -> Select an IQN in the list -> The login button is activated and is Clear

Moving to VERIFIED!

Added Screen shot of solution!

Comment 9 Kevin Alon Goldblatt 2015-07-08 15:17:30 UTC
Created attachment 1049909 [details]
screenshot

Added screen shot of solution

Comment 11 errata-xmlrpc 2016-03-09 20:39:51 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHEA-2016-0376.html