Skip to content

fix: apply nav-indent left padding correctly#6001

Merged
puikinsh merged 1 commit intoColorlibHQ:masterfrom
Kneemund:master
Mar 18, 2026
Merged

fix: apply nav-indent left padding correctly#6001
puikinsh merged 1 commit intoColorlibHQ:masterfrom
Kneemund:master

Conversation

@Kneemund
Copy link
Copy Markdown
Contributor

In the latest version of AdminLTE v4 (rc7), the nav-indent behaves unexpectedly. The SCSS selector .nav-indent & is compiled to .nav-indent .sidebar-menu .nav-treeview.

That means that you would have to apply nav-indent to a parent of the sidebar menu (e.g. the sidebar wrapper), which is very unintuitive compared to the other nav classes.

In compact mode, the selector for the equivalent rule is simply .nav-compact.nav-indent .nav-treeview, which means that .nav-indent works differently (and as expected) there.

This PR makes the problematic non-compact mode rule behave the same as the other nav-indent rules.

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 13, 2026

Deploy Preview for adminlte-v4 failed.

Name Link
🔨 Latest commit f4c02eb
🔍 Latest deploy log https://app.netlify.com/projects/adminlte-v4/deploys/69b409d3cf015d00089ec5dc

@puikinsh
Copy link
Copy Markdown
Member

Thanks @Kneemund! Great catch on the selector scoping — the .nav-indent & nesting was definitely unintuitive compared to how .nav-compact.nav-indent works. Appreciate you keeping things consistent. Merging this in. 👍

@puikinsh puikinsh merged commit 2d9e2ce into ColorlibHQ:master Mar 18, 2026
6 of 10 checks passed
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.

2 participants