Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show cable colors on device page #3457

Closed
Bitsky opened this issue Aug 27, 2019 · 2 comments · Fixed by #3673
Closed

Show cable colors on device page #3457

Bitsky opened this issue Aug 27, 2019 · 2 comments · Fixed by #3673
Labels
status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application

Comments

@Bitsky
Copy link

Bitsky commented Aug 27, 2019

Environment

  • Python version: 2.7.5
  • NetBox version: 2.6.1

Proposed Functionality

Show the colors of cables on the interfaces list of a device. Right now it's just a blue dash after the cable number/name. To make the color stand out better, maybe change eg "#123 -" into buttons (like "[#123]") similar to those [Primary]/[Active] ones in the Status/Role column. Besides, because most cables are gray by default, undefined colors should fallback to gray instead of blue; or have the button transparent with only the "#123" text.

Use Case

When you work in the rack, you want a quick overview of the device you're working on. Right now you have to click on every cable to see its color.

Database Changes

None

External Dependencies

None

@jeremystretch
Copy link
Member

because most cables are gray by default, undefined colors should fallback to gray instead of blue

One cannot assume what the most popular cable color is, and there is no default cable color in NetBox.

@Bitsky
Copy link
Author

Bitsky commented Sep 7, 2019

Well, that's why I added:

or have the button transparent with only the "#123" text

If you consider the current dash to somehow relate to cables and their colors, blue would be less common than gray. But that's beside the point. It should be an easy change which increases the general overview of a device.

@jeremystretch jeremystretch added status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application labels Oct 15, 2019
jeremystretch added a commit that referenced this issue Nov 6, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Feb 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants