Github user necouchman commented on a diff in the pull request:

    https://github.com/apache/guacamole-manual/pull/92#discussion_r210519471
  
    --- Diff: src/chapters/configuring.xml ---
    @@ -3009,13 +3009,102 @@ ed272546-87bd-4db9-acba-e36e1a9ca20a
                                                 <primary>SSH</primary>
                                                 <secondary>color 
scheme</secondary>
                                             </indexterm>The color scheme to 
use for the terminal
    -                                        emulator used by SSH connections. 
Each color scheme dictates
    -                                        the default foreground and 
background color for the
    -                                        terminal. Programs which specify 
colors when printing text
    -                                        will override these 
defaults.</para>
    -                                    <para>This parameter is optional. By 
default, Guacamole will
    -                                        render text as gray over a black 
background.</para>
    -                                    <para>Possible values are:</para>
    +                                        emulator used by SSH connections. 
It consists of a
    +                                        semicolon-separated series of 
name-value pairs, separated by
    --- End diff --
    
    I'd suggest a slight tweak to this language - perhaps something like:
    
    > It consists of a semicolon-separated series of name-value pairs, where 
each name-value pair is separate by a colon, ...
    
    or something else that clarifies this a little bit.  My initial take on 
this is, "Well, is it separate by a semicolon or a colon?"  I get it when I 
look at the examples, but I think it has the potential to cause some confusion.


---

Reply via email to