Skip to content

[BootstrapAdminUi] Remove additional spacing in sidebar menu - #405

Merged
loic425 merged 1 commit into
Sylius:mainfrom
loic425:bootstrap-admin-ui-remove-menu-min-height
Nov 12, 2026
Merged

loic425 merged 1 commit into
Sylius:mainfrom
loic425:bootstrap-admin-ui-remove-menu-min-height

Conversation

@loic425

@loic425 loic425 commented Nov 12, 2026 •

Copy link
Copy Markdown
Member
Q A
Branch? main
Bug fix? yes
New feature? no
BC breaks? no
License MIT

Follow-up of #404. The sidebar submenu items look more spaced out than in the Sylius admin.

Sylius removed min-height: 44px from .navbar-collapse a.nav-link / a.dropdown-item in Sylius/Sylius@7b2d5a77e7 ("Remove additional spacings in admin sidebar menu"), but the rule was kept in BootstrapAdminUi.

  • Submenu items (.dropdown-item) go from 44px to 30px, same as in the Sylius admin.
  • Top-level items are unchanged (Tabler already sets a 40px min-height on them).

public/ assets were rebuilt with yarn build.

Before After
Sidebar menu before Sidebar menu after

@loic425
loic425 merged commit 78eb1cb into Sylius:main Nov 12, 2026
5 checks passed
@loic425
loic425 deleted the bootstrap-admin-ui-remove-menu-min-height branch November 12, 2026 08:50
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.

1 participant