Bug #5905


Hard coded background colors on status_gateways.php

Added by Jared Dillard over 8 years ago. Updated over 8 years ago.

Jared Dillard
Web Interface
Target version:
Start date:
Due date:
% Done:


Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
Affected Architecture:


I noticed hardcoded background colors on status_gateways.php that haven't been converted to bootstrap classes (success, warning, etc) yet.

I did a grep for the color below and it showed some possible places where they might be hiding.

grep -rn "90EE90" returns:

/src/usr/local/www/includes/ $bgcolor = "#90EE90"; // lightgreen
/src/usr/local/www/status_gateway_groups.php:68: define('LIGHTGREEN', '#90EE90');
/src/usr/local/www/status_gateways.php:67: define('LIGHTGREEN', '#90EE90');
/src/usr/local/www/status_lb_pool.php:74: define('LIGHTGREEN', '#90EE90');
/src/usr/local/www/status_lb_vs.php:65: define('LIGHTGREEN', '#90EE90');

Actions #1

Updated by Jim Thompson over 8 years ago

  • Assignee set to Anonymous
  • Target version changed from 2.3 to 2.4.0
  • Affected Version set to 2.3
Actions #2

Updated by Anonymous over 8 years ago

I think I made those defines about a year ago. Problem is Bootstrap gives us only four colors and they are fairly dense/saturated. The pastel colors used in 2.2.x just seemed better suited to shading a table cell. What do you think?

We could substitute class names and CSS ifthat makes people feel better.

Actions #3

Updated by Anonymous over 8 years ago

  • Status changed from New to Feedback
  • Assignee changed from Anonymous to Jared Dillard
  • Priority changed from Normal to Low
Actions #4

Updated by Jared Dillard over 8 years ago

  • Status changed from Feedback to Resolved

Fixed the following files:


Actions #5

Updated by NOYB NOYB over 8 years ago

Revision 3b53c227
Added by Jared Dillard about 9 hours ago

Ticket #5905 remove unused code

That commit breaks the system information widget in at least 2 places.
Although the initial page loads fine. When it auto refreshes the CPU Type has extra line of info that appears to be CPU load, and state table size % is undefined.

Actions #6

Updated by Chris Buechler over 8 years ago

  • Target version changed from 2.4.0 to 2.3

He just missed shifting the values to account for the removal of [8], fixed now.


Also available in: Atom PDF