Xymon Mailing List Archive search

Color Status GIF for host status page

list Jeremy Laidman
Tue, 29 Sep 2015 11:04:41 +1000
Message-Id: <user-ccb04aac3635@xymon.invalid>

On 26 September 2015 at 12:12, Rob Steuer <user-dc5347ce617b@xymon.invalid> wrote:
      $DOTCOLOR Domain $DOMAIN expires on: $EXPIRESTRING

Rob, that's great work.  A couple of suggestions.  Firstly, in some cases
you want the color to be hard up against the next word with no spaces, and
so you need to include a semicolon, like so:

&red;Bad widget smell at 4o'clock

Secondly, you could streamline your code by removing all of the DOTCOLOR=
lines and instead use this at the end:

&$COLOR Domain $DOMAIN expires on: $EXPIRESTRING

This prunes 3 lines of code, and eliminates one variable, with no loss in
readability (IMHO).