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

No search bar on mobile #16273

Closed
ChrisRecioHive opened this issue May 23, 2024 · 5 comments · Fixed by #16672
Closed

No search bar on mobile #16273

ChrisRecioHive opened this issue May 23, 2024 · 5 comments · Fixed by #16672
Assignees
Labels
severity: low Does not significantly disrupt application functionality, or a workaround is available status: accepted This issue has been accepted for implementation topic: UI/UX User interface or user experience related work type: bug A confirmed report of unexpected behavior in the application

Comments

@ChrisRecioHive
Copy link

Deployment Type

Self-hosted

NetBox Version

v4.0.2

Python Version

3.12

Steps to Reproduce

  1. Open Netbox on a mobile device

Expected Behavior

Display a search bar either at the top or in the collapsible menu

Observed Behavior

No search bar

image

@ChrisRecioHive ChrisRecioHive added status: needs triage This issue is awaiting triage by a maintainer type: bug A confirmed report of unexpected behavior in the application labels May 23, 2024
@amit177
Copy link
Contributor

amit177 commented May 23, 2024

The reason it doesn't show up is that the container has the d-none and d-lg-flex classes, causing the search bar to show only on big screens.

Changing d-lg-flex to d-sm-flex should fix the issue, but would look weird.
Perhaps the better solution might be moving the search bar to the top of the menu when it is expanded.

@jeremystretch jeremystretch removed their assignment May 24, 2024
@jeremystretch jeremystretch added status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation complexity: low Requires minimal effort to implement topic: UI/UX User interface or user experience related work and removed status: needs triage This issue is awaiting triage by a maintainer labels May 24, 2024
@ITJamie
Copy link
Contributor

ITJamie commented May 27, 2024

related: there's no darkmode / lightmode switch on mobile ether in the top bar.

@jeremystretch jeremystretch added severity: low Does not significantly disrupt application functionality, or a workaround is available and removed complexity: low Requires minimal effort to implement labels May 30, 2024
@ljarasius
Copy link
Contributor

I can take this one @jeremystretch

@jeremystretch jeremystretch added status: accepted This issue has been accepted for implementation and removed status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation labels May 31, 2024
@remram44
Copy link

Is there a workaround for this? Other than tapping the URL bar and adding /search I mean.

@pv2b
Copy link
Contributor

pv2b commented Jun 12, 2024

Is there a workaround for this? Other than tapping the URL bar and adding /search I mean.

You could add a Note widget to your dashboard with the following contents:

# [Search](/search)

This will make a heading-sized link with the word "search" you can click on from the dashboard to get to the search page. Heading-size so that it's a little easier to tap on mobile. Of course it'll also show up from your dashboard on desktop.

This way you can just go to the dashboard and have a search link from there. Not ideal but I think that's the best you can do without any code changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
severity: low Does not significantly disrupt application functionality, or a workaround is available status: accepted This issue has been accepted for implementation topic: UI/UX User interface or user experience related work type: bug A confirmed report of unexpected behavior in the application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants