anmolbabu has posted comments on this change.

Change subject: webadmin : Volume Capacity column in volume tab
......................................................................


Patch Set 17:

(1 comment)

....................................................
File 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationTemplates.java
Line 35:                     "<div style='background: {2}; width: {0}%; height: 
100%'></div>" +
Line 36:                     "<div class='engine-progress-text'>{1}</div></div>"
Line 37:                     +
Line 38:                     "<div>{3}</div>")
Line 39:             SafeHtml progressBarWithBottomText(Integer progress, 
String text, String color, String sizeString);
It displays result in 2 lines one within progress bar and the other below 
it.Actually the first 3 divs I have just copied them from the RHEVM code for 
displaying the similar progress bar in Hosts tab for cpu.The RHEVM code is just 
above my change.Yes,but the design now I need to change to accommodate the 
refresh button.And I wanted to know 1 more thing i.e,to refresh a particular 
volume capacity,we need to explicitly invoke the query right?
Line 40: 
Line 41:     /**
Line 42:      * Creates a tree-item HTML
Line 43:      *


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I04151d78838c1398cff42c84104e21d61d9c7747
Gerrit-PatchSet: 17
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: anmolbabu <anb...@redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayi...@redhat.com>
Gerrit-Reviewer: Ramesh N <rnach...@redhat.com>
Gerrit-Reviewer: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtri...@redhat.com>
Gerrit-Reviewer: anmolbabu <anb...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to