ColorlibHQ / AdminLTE

AdminLTE - Free admin dashboard template based on Bootstrap 5
https://adminlte.io
MIT License
44.11k stars 18.17k forks source link

[BUG] v4.x main sidebar icon, text spacing #4140

Open umer936 opened 2 years ago

umer936 commented 2 years ago

We've been using the 4.x branch of AdminLTE and found that on the sidebar, the icon does not keep a fixed width. This means icons of different sizes make the link text not be aligned. I was able to trace it to this CSS property that is in AdminLTE 3.1 (main-sidebar.scss:123), but not in AdminLTE 4:

.nav-sidebar > .nav-item .nav-icon {
    width: 1.6rem;
}

I went ahead and added it to my own CSS file, but it should probably be in the main-sidebar.scss (or nav-sidebar.scss) file for AdminLTE 4.

umer936 commented 2 years ago

It looks like it could go in nav-sidebar.scss:34. I can make a PR if that's where it should go.

danny007in commented 2 years ago

PR are always welcome 😃

danny007in commented 1 year ago

is this issue still exist ?

umer936 commented 1 year ago

Good question - I forgot about this. I haven't updated to the latest v4 / latest Bootstrap to see if it still requires that css property. I'll see if I can get to that before next week.