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

Return the slim sidebar menu #15614

Open
julianstolp opened this issue Apr 4, 2024 · 14 comments
Open

Return the slim sidebar menu #15614

julianstolp opened this issue Apr 4, 2024 · 14 comments
Assignees
Labels
complexity: medium Requires a substantial but not unusual amount of effort to implement status: accepted This issue has been accepted for implementation topic: UI/UX User interface or user experience related work type: feature Introduction of new functionality to the application

Comments

@julianstolp
Copy link

NetBox version

v4.0-beta1

Feature type

Change to existing functionality

Proposed functionality

The new sidebar is wider and there is no possiblity to hide this massive sidebar.
Proposed change:
Some sticky icon like in the old UI
image
image

Use case

Use the screen more efficiently and prevent/minimize horizontal scrolling in tables.

Database changes

External dependencies

@julianstolp julianstolp added status: needs triage This issue is awaiting triage by a maintainer type: feature Introduction of new functionality to the application labels Apr 4, 2024
@jeremystretch jeremystretch changed the title [Beta] Return the slim sidebar menu Return the slim sidebar menu Apr 4, 2024
@jeremystretch jeremystretch added status: under review Further discussion is needed to determine this issue's scope and/or implementation and removed status: needs triage This issue is awaiting triage by a maintainer labels Apr 4, 2024
@jeremystretch jeremystretch removed their assignment Apr 4, 2024
@jeremystretch
Copy link
Member

IMO the collapsed navigation menu in v3 is unnecessary and, when not stickied, rather annoying. I've been using the current UI for months on a relatively narrow (1280px) viewport without issue. I see no reason to retain the old functionality, but will leave this open for discussion if someone else would like to own it.

@julianstolp
Copy link
Author

julianstolp commented Apr 4, 2024

We have almost exclusively used the narrow navigation bar to display as many columns as possible. It would be nice if this option were available again.

@jeremystretch jeremystretch added beta Concerns a bug/feature in a beta release topic: UI/UX User interface or user experience related work labels Apr 4, 2024
@sleepinggenius2
Copy link
Contributor

I also would like to see the collapsed navigation bar return. The new one is 33px wider than even the fully expanded one before and 241px wider than the collapsed one. When you are displaying a table with a lot of columns, every pixel helps. The worst offender for me is usually the Interfaces table, where I find it difficult to fit all the columns I need even on a 2560px wide screen.

I can tell you from experience with my end users that they have always been thrilled when I show them the ability to collapse the menu and get some valuable screen real estate back. From observation, the menu is almost never used in our environment, so even the collapsed version has been identified as an annoyance. The typical flow I see is users finding what they are looking for through the global search, then navigating around from there through breadcrumbs or other links within the view. In fact, most of my users don't even know where to find things in the menu because of how infrequently they use it.

@petersonbasso
Copy link

The collapsed navigation menu is very useful, especially for small screens.
Better to keep the option and leave it up to the user to keep it fixed.

@jeremystretch
Copy link
Member

Opening this for a volunteer to implement it.

@jeremystretch jeremystretch added status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation and removed status: under review Further discussion is needed to determine this issue's scope and/or implementation labels Apr 8, 2024
@julianstolp
Copy link
Author

julianstolp commented Apr 9, 2024

If someone is able to implement this, there is an open FR in the Tabler repository. It would certainly be good to implement this there so that no/less custom code is necessary in NetBox.

tabler/tabler#1803

@heroin-moose
Copy link

Please, please, please, please, please, please, please, please, please bring it back.

@jeremystretch
Copy link
Member

@heroin-moose would you like to take ownership of this issue?

@heroin-moose
Copy link

heroin-moose commented May 14, 2024

@jeremystretch, yeah, why not. Would you prefer it solved via tabler/tabler#1803?

@jeremystretch
Copy link
Member

Upstream support would be strongly preferred, but of course that's out of our control. We'd only accept a local implementation if it can be achieved in a clean, maintainable manner with minimal additional code.

@heroin-moose
Copy link

I'll do my best.

@jeremystretch jeremystretch added status: accepted This issue has been accepted for implementation complexity: medium Requires a substantial but not unusual amount of effort to implement and removed status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation labels May 14, 2024
@bobbwest
Copy link

An interim solution would be to have a toggle that would disable the responsive full side navbar menu, keeping the dropdown navbar menu that is usually activated on narrow viewports. At least that means people wanting a wide tabular display can achieve that, and still maintain some access to the navbar menu.

@jeremystretch
Copy link
Member

@heroin-moose are you still planning to work on this, or should we release it for another volunteer?

@heroin-moose
Copy link

heroin-moose commented May 29, 2024

I'm slowly getting there, but if someone gets there first I would gladly pass the issue. I have a PoC that mostly works, I will fix the remaining issues I see and submit it to Tabler.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: medium Requires a substantial but not unusual amount of effort to implement status: accepted This issue has been accepted for implementation topic: UI/UX User interface or user experience related work type: feature Introduction of new functionality to the application
Projects
None yet
Development

No branches or pull requests

6 participants