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

Fix hidden navbar buttons and icon size #1064

Merged
merged 7 commits into from
Feb 18, 2023
Merged

Fix hidden navbar buttons and icon size #1064

merged 7 commits into from
Feb 18, 2023

Conversation

co-decode
Copy link
Contributor

@co-decode co-decode commented Feb 17, 2023

Fixes #1063

  • I have removed all instances of the 'hidden' class in the navbar. It was colliding with bootstrap.
  • I have replaced the 'hidden' class with a custom css class, 'nav-hidden', in navbar.css
  • I have made navbar icon size specifications !important.

I haven't been able to test this without DEBUG on.
I can't work out how to turn the DEBUG variable to False.
I am modifying the blt/settings.py file but I seem to be missing something.
Any help would be appreciated.

@co-decode co-decode closed this by deleting the head repository Feb 18, 2023
@co-decode co-decode reopened this Feb 18, 2023
@fredfalcon fredfalcon added this pull request to the merge queue Feb 18, 2023
Merged via the queue into OWASP-BLT:main with commit 6802006 Feb 18, 2023
DonnieBLT pushed a commit that referenced this pull request Aug 12, 2023
* updating main origin

* Revert "updating main origin"

This reverts commit a996038.

Linter updated all py files inadvertently

* fixed navbar icon size, and button display
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Navbar elements are hidden above 640px
2 participants