Descending rack numbering & space calculation #526

Closed
opened 2025-12-29 16:22:53 +01:00 by adam · 3 comments
Owner

Originally created by @j-boy on GitHub (Nov 10, 2016).

For version 1.7.0

When adding a device to a rack, the GUI says "The lowest-numbered unit occupied by the device" but it makes more sense for the user to enter the (physical) lowest position.
For racks with descending numbering, this will be the highest rack number, for racks with ascending numbering it will be the lowest rack number.
A little bit more complexity in the code for a more consistent user experience.

Originally created by @j-boy on GitHub (Nov 10, 2016). For version 1.7.0 When adding a device to a rack, the GUI says "The lowest-numbered unit occupied by the device" but it makes more sense for the user to enter the (physical) lowest position. For racks with descending numbering, this will be the highest rack number, for racks with ascending numbering it will be the lowest rack number. A little bit more complexity in the code for a more consistent user experience.
adam added the type: feature label 2025-12-29 16:22:53 +01:00
adam closed this issue 2025-12-29 16:22:53 +01:00
Author
Owner

@jeremystretch commented on GitHub (Nov 11, 2016):

I don't see that it makes much difference to be honest, but I'm happy to keep this issue open if you're interested in submitting a pull request.

@jeremystretch commented on GitHub (Nov 11, 2016): I don't see that it makes much difference to be honest, but I'm happy to keep this issue open if you're interested in submitting a pull request.
Author
Owner

@LBegnaud commented on GitHub (Nov 11, 2016):

It doesn't matter to me either way, but I figured I'd chime in to remind whoever does the PR that converting a rack from one type to the other needs to have some extra logic to account for this if it does change.

@LBegnaud commented on GitHub (Nov 11, 2016): It doesn't matter to me either way, but I figured I'd chime in to remind whoever does the PR that converting a rack from one type to the other needs to have some extra logic to account for this if it does change.
Author
Owner

@jeremystretch commented on GitHub (Dec 15, 2016):

Closing due to lack of activity.

@jeremystretch commented on GitHub (Dec 15, 2016): Closing due to lack of activity.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#526