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

Adjust status bar and top menu #2813

Merged
merged 2 commits into from
Dec 5, 2024
Merged

Adjust status bar and top menu #2813

merged 2 commits into from
Dec 5, 2024

Conversation

jevenski
Copy link
Contributor

@jevenski jevenski commented Dec 3, 2024

This patch contains some unnoticeable changes by non-developers. It serves mainly for a clean-up purpose.

  • Change status bar position to static because height of the navbar, the main content and the status bar has been set to be calculated dynamically according to the window height.
  • Define status bar border color and background color using css variables, with CSS gradient for background in use in some themes (Excel, Oblivion).
  • Remove verbose utility class names on status bar.
  • Remove unused outdated CSS rules.
  • Set a max height on the collapsible navbar menu to fit in some extreme situations where one might have a screen with small height or a collapsible menu with a lot of buttons.

@jevenski jevenski force-pushed the status-bar branch 3 times, most recently from 28d24a0 to c938023 Compare December 3, 2024 10:32
- Change status bar position to static because height of the navbar,
  the main content and the status bar has been set to be calculated
  dynamically according to the window height.
- Define status bar border color and background color using css
  variables, with CSS gradient for background in use in some themes
  (Excel, Oblivion).
- Remove verbose utility class names on status bar.
- Remove unused outdated CSS rules.
- Set a max height on the collapsible navbar menu to fit in some
  extreme situations where one might have a screen with small height
  or a collapsible menu with a lot of buttons.
@stickz
Copy link
Collaborator

stickz commented Dec 4, 2024

There is a problem with webui resizing introduced by this pull request.

Resize web browser window to mobile and open the topmenu. Keep it open and resize back to desktop.
white_background

The background for the menu items on desktop changes to white. The topmenu for mobile remains open on resize.

- Fix top menu collapsing when opening dialog windows on mobile.
- Fix top menu not collapsing when switching from mobile to desktop.
@jevenski
Copy link
Contributor Author

jevenski commented Dec 5, 2024

This is an old bug introduced by an earlier commit. The new commit also fixes this issue: #2814

@stickz stickz added the bug label Dec 5, 2024
@stickz stickz linked an issue Dec 5, 2024 that may be closed by this pull request
4 tasks
@stickz stickz merged commit 036d8a9 into Novik:develop Dec 5, 2024
@jevenski jevenski deleted the status-bar branch December 5, 2024 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Collapsible top nav bar opens when dialog is opened
2 participants