sp-techdocs commented on code in PR #460:
URL: 
https://github.com/apache/cloudstack-documentation/pull/460#discussion_r1892048238


##########
source/adminguide/host_and_storage_tags.rst:
##########
@@ -31,30 +31,39 @@ There are two types of host tags:
 To explain the behavior of host tags, some examples will be demonstrated with 
two hosts (Host1 and Host2):
 
 #. Tag setup:
+
     * Host1: h1
     * Host2: h2
     * Offering: h1
+
     When a VM is created with the offering, the deployment will be carried out 
on Host1, as it is the one that has the tag compatible with the offering.

Review Comment:
   The idea for indenting this paragraph was to make it a child element of the 
parent "#. Tag setup:" numbered list item. In the same way as the bulleted list 
above it is indented. This way both child elements look like belonging to the 
numbered list item.
   
   For me, removing the indentation for this paragraph would lead to showing it 
as being at the same logical level as the numbered list item, looking like it 
is not related to it. And they are actually related.
   
   Overall, it seems the four "#. Tag setup:" numbered list items here are not 
related, and it might be better to convert them to bulleted list items. While 
keeping their child content indented.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to