Skip to content

Left menu tweak#24249

Open
chippison wants to merge 7 commits into5.x-devfrom
left-menu-tweak
Open

Left menu tweak#24249
chippison wants to merge 7 commits into5.x-devfrom
left-menu-tweak

Conversation

@chippison
Copy link
Contributor

@chippison chippison commented Mar 19, 2026

Description

DEV-19960

We want to tweak our left menu so that it looks better.
We also wanted to move the search bar (QuickAccess) into the left menu instead of it inside our top_controls.

For this, we also changed the structure so that our left menu, is outside of the page element and into its own so that we can make this move to the top, just under the main header (logo).

Checklist

  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules

Review

@chippison chippison added this to the 5.9.0 milestone Mar 19, 2026
@chippison chippison force-pushed the left-menu-tweak branch 17 times, most recently from 5ba02ae to f7ee52c Compare March 25, 2026 00:48
@chippison chippison marked this pull request as ready for review March 25, 2026 01:06
…e 'top_controls' and 'page' stack on top of each other beside left-menu
 - changing UI of search bar based on spec
 - transferring the css that controls 'looks' into QuickAccess.vue
 - fix screenshot issue
 - updating TagManager
 - fixing for multi-line menu items
 - update based on feedback
@chippison chippison force-pushed the left-menu-tweak branch 3 times, most recently from 9774455 to 31ad0f3 Compare March 26, 2026 00:14
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have made the left menu span the whole height of the page this is why the new menu is really long on this page (Manage Plugins)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Search menu is no longer here since we transferred it on the left side menu

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We also made the search results dropdown scrollable so that we do not get a very long search result dropdown

<div class="layoutWithSidebarContent">
<div class="top_controls">
{% if hideQuickAccess is not defined or not hideQuickAccess %}
<div class="quickAccessPlaceholder" aria-hidden="true"></div>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needed to add this 'placeholder' here because when I removed the QuickAccess Component, it had a weird issue in the PeriodSelector screenshot went a bit off. I tried w/codex to fix this, but was not really successful and I did not want to spend too much time on this as the UI is still changing a lot.

I'll try to figure this out more when the UI tweaks have settled down.

Image

@chippison chippison requested a review from a team March 26, 2026 03:11
@chippison
Copy link
Contributor Author

We will not merge this issue until all other UI Tweaks are done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant