Alissa Bonas has posted comments on this change.

Change subject: webadmin: move-copy disks - inactive domains indication
......................................................................


Patch Set 1: (1 inline comment)

....................................................
File 
frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Constants.java
Line 802: 
Line 803:     @DefaultStringValue("Source Storage Domain is not active")
Line 804:     String sourceStorageDomainIsNotActiveMsg();
Line 805: 
Line 806:     @DefaultStringValue("No active target Storage Domain is 
available")
There is no need to write in capital letters "storage domain" in the middle of 
the sentence (in any of the messages, I am just commenting one of them not to 
repeat the comment)
Line 807:     String noActiveTargetStorageDomainAvailableMsg();
Line 808: 
Line 809:     @DefaultStringValue("No active source Storage Domain is 
available")
Line 810:     String noActiveSourceStorageDomainAvailableMsg();


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

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

Reply via email to