Description
NetBox version
v3.7.3
Feature type
Data model extension
Proposed functionality
This is revisit of #13264
Currently the background color of an interface is like this :
disabled: red
virtual: blue
enabled, marked_connected: green
enabled, connected (cable status color) : green/blue/..
enabled, no cable : white
def get_cabletermination_row_class(record):
if record.mark_connected:
return 'success'
elif record.cable:
return record.cable.get_status_color()
return ''
def get_interface_row_class(record):
if not record.enabled:
return 'danger'
elif record.is_virtual:
return 'primary'
return get_cabletermination_row_class(record)
My proposal is to change the color coding to:
disabled: white
virtual: blue
enabled, marked_connected: green
enabled, connected (cable status color) : green/blue/...
enabled, no cable : red
def get_cabletermination_row_class(record):
if record.mark_connected:
return 'success'
elif record.cable:
return record.cable.get_status_color()
return 'danger'
def get_interface_row_class(record):
if not record.enabled:
return ''
elif record.is_virtual:
return 'primary'
return get_cabletermination_row_class(record)
Use case
The 'red' color is confusing, as it tells me something is 'wrong'. but having an interface in the disable state is not wrong.
vs
Having an 'enabled' interface, without a cable is 'danger'
A disabled interface is just good, that can be white.
Can we have a poll with the community to see what they prefer ?
Database changes
No response
External dependencies
No response