Alissa Bonas has posted comments on this change.

Change subject: webadmin: Hide V Format field in storage domain
......................................................................


Patch Set 1: Fails

following Daniel's comment, it's problematic to hide the format field for the 
following reasons:
1. for uninitialized data center, the format is not shown next to dc name.
2. though 1 can be taken care of by displaying the format field in that 
case,there is another problem - if the data center name will be too long, the 
format in the data center names dropdown list will be cut and not be shown, 
thus the user will miss that info.
so a better representation solution should be considered.

--
To view, visit http://gerrit.ovirt.org/15323
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4478581f9526635ed8f9cc5ee922a544dc530e87
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alissa Bonas <[email protected]>
Gerrit-Reviewer: Alissa Bonas <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to