|
From: | lemzwerg |
Subject: | Re: Doc: NR - Doc CSS colour support (issue 551270043 by address@hidden) |
Date: | Sat, 21 Dec 2019 07:34:57 -0800 |
LGTM, thanks! https://codereview.appspot.com/551270043/diff/583260043/Documentation/notation/editorial.itely File Documentation/notation/editorial.itely (right): https://codereview.appspot.com/551270043/diff/583260043/Documentation/notation/editorial.itely#newcode576 Documentation/notation/editorial.itely:576: X11 and CSS colors will not always be exactly the same as a similarly Wouldn't it be better to use present tense? At least this is the recommendation in the GNU coding standards, see https://www.gnu.org/prep/standards/html_node/GNU-Manuals.html https://codereview.appspot.com/551270043/diff/583260043/Documentation/notation/editorial.itely#newcode584 Documentation/notation/editorial.itely:584: Hex codes for either CSS or X11 cannot be used. s/hex codes/hexadecimal values/ https://codereview.appspot.com/551270043/
[Prev in Thread] | Current Thread | [Next in Thread] |