Hard coded background colors on status_gateways.php
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/functions.inc.php:79: $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');
Updated by Steve Beaver over 5 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.
Updated by Jared Dillard over 5 years ago
- Status changed from Feedback to Resolved
Fixed the following files:
Updated by NOYB NOYB over 5 years ago
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.