Skip to content

Commit 61cf1d4

Browse files
committed
sidebar fixes
fixed issue with missing first menu icon. fixed issue with logo separator still showing when collaped and no logo submitted
1 parent 9100ceb commit 61cf1d4

File tree

4 files changed

+10
-2
lines changed

4 files changed

+10
-2
lines changed

src/css/bundle.css

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4442,6 +4442,10 @@ html:has(.kroma-navbar) .kroma-sidebar{
44424442
background: var(--neutral-300);
44434443
}
44444444

4445+
.kroma-sidebar.collapsed .sdb-logo:not(:has(img)):after {
4446+
display: none;
4447+
}
4448+
44454449
.kroma-sidebar .sdb-logo span {
44464450
margin: auto 0;
44474451
}

src/css/components/sidebar.css

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -142,6 +142,10 @@ html:has(.kroma-navbar) .kroma-sidebar{
142142
background: var(--neutral-300);
143143
}
144144

145+
.kroma-sidebar.collapsed .sdb-logo:not(:has(img)):after {
146+
display: none;
147+
}
148+
145149
.kroma-sidebar .sdb-logo span {
146150
margin: auto 0;
147151
}

src/js/bundle.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1509,7 +1509,7 @@ export class KromaSidebar {
15091509
page.text = a.innerText ?? 'Page';
15101510
page.title = page.text;
15111511
page.icon = a.previousElementSibling;
1512-
if( (page.icon.tagName ?? '') != 'I' ){
1512+
if(!page.icon || (page.icon.tagName ?? '') != 'I' ){
15131513
page.icon = document.createElement('i');
15141514
page.icon.classList.add('fa-solid');
15151515
page.icon.classList.add('fa-link');

src/js/components/sidebar.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ export class KromaSidebar {
5555
page.text = a.innerText ?? 'Page';
5656
page.title = page.text;
5757
page.icon = a.previousElementSibling;
58-
if( (page.icon.tagName ?? '') != 'I' ){
58+
if(!page.icon || (page.icon.tagName ?? '') != 'I' ){
5959
page.icon = document.createElement('i');
6060
page.icon.classList.add('fa-solid');
6161
page.icon.classList.add('fa-link');

0 commit comments

Comments
 (0)