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

Rack Elevation Sorting Enhancement #9665

Closed
NavyWings opened this issue Jul 5, 2022 · 2 comments · Fixed by #10233
Closed

Rack Elevation Sorting Enhancement #9665

NavyWings opened this issue Jul 5, 2022 · 2 comments · Fixed by #10233
Assignees
Labels
status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application

Comments

@NavyWings
Copy link

NetBox version

v3.2.2

Feature type

Change to existing functionality

Proposed functionality

Would be a helpful feature if we were able to sort the rack elevation by rack facility ID.

Currently, the rack elevation view sorts only on rack name. It can be sorted in "normal" (numerically ascending) or "reversed" (numerically descending.)"

Use case

Rack naming conventions are dependent on organization, not all entities name or number racks in a contiguous linear fashion from left to right. In our case, our center racks are numbered racks one and two and numbering increases as you move further from the center racks. It would be a helpful feature to see the rack elevation ordered by facility ID from left to right while retaining a custom rack naming at the top.

Database changes

Likely none, but unknown.

External dependencies

None

@NavyWings NavyWings added the type: feature Introduction of new functionality to the application label Jul 5, 2022
@jeremystretch jeremystretch added the status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation label Jul 27, 2022
@arthanson
Copy link
Collaborator

@jeremystretch The current sort is a button-toggle, which wouldn't work for adding any new sorting. If we add this I see two potential solutions:

  1. Add a "Sort By" dropdown in-place of the buttons, with options "Name (A-Z)", "Name (Z-A)", "Facility ID (A-Z)", "Facility ID (Z-A)" could also use (Ascending) / (Descending) instead of (A-Z).
  2. Move the button below the tabs and have a "Sort By: " and then the dropdown showing the current sorting. (see screenshot)

I looked around and all the other sorting is done by column headers, so this looks like a sort of one-off UI.

sortby

@jeremystretch
Copy link
Member

I think a dropdown with all four combinations makes sense. A-Z and Z-A is fine.

@arthanson arthanson self-assigned this Aug 31, 2022
@arthanson arthanson removed the status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation label Aug 31, 2022
@jeremystretch jeremystretch added the status: accepted This issue has been accepted for implementation label Sep 1, 2022
arthanson added a commit that referenced this issue Sep 1, 2022
arthanson added a commit that referenced this issue Sep 1, 2022
jeremystretch added a commit that referenced this issue Sep 2, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 2, 2022
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.

3 participants