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

    https://github.com/apache/flink/pull/3751#discussion_r115753699
  
    --- Diff: docs/page/css/syntax.css ---
    @@ -77,3 +77,6 @@
     .highlight .vg { color: #008080 } /* Name.Variable.Global */
     .highlight .vi { color: #008080 } /* Name.Variable.Instance */
     .highlight .il { color: #009999 } /* Literal.Number.Integer.Long */
    +
    +/* Missing YAML styles */
    +.highlight .l-Scalar-Plain { color: #008080 }
    --- End diff --
    
    Yeah, the highlighter we use doesn't have any built-in styling for YAML. I 
had to search around quite a bit for context and came up mostly dry.
    
    Moreover, the actual tokenizer it uses only emits two token types, 
l-Scalar-Plain and p-Indicator. (You can see this in action by inspecting the 
HTML 
[here](https://ci.apache.org/projects/flink/flink-docs-release-1.2/monitoring/metrics.html#jmx-orgapacheflinkmetricsjmxjmxreporter))
    
    This change applies the color I found most applicable from the existing 
highlighting rules and applies it to l-Scalar-Plain to make it at least 
slightly more interesting than black-on-white.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to