Skip to content

Commit 274ddd9

Browse files
fix: include right-arrow icon in hover background; remove margin, rounded corners, and clean unrelated styles as per review
1 parent f4c15ab commit 274ddd9

File tree

2 files changed

+6
-10
lines changed

2 files changed

+6
-10
lines changed

packages/ui-components/src/Containers/Sidebar/SidebarItem/index.module.css

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,10 @@
88
w-full
99
items-center
1010
overflow-hidden
11-
rounded-md
1211
text-sm
1312
text-neutral-800
1413
dark:text-neutral-200;
1514

16-
margin-bottom: 1px;
17-
1815
&:hover {
1916
&:not(.progression) .label {
2017
@apply rounded-sm
@@ -33,11 +30,7 @@
3330

3431
.label {
3532
@apply font-regular
36-
p-2
37-
text-sm
38-
transition-colors
39-
duration-200
40-
ease-in-out;
33+
p-2;
4134
}
4235

4336
.progressionIcon {

packages/ui-components/src/Containers/Sidebar/SidebarItem/index.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,12 @@ const SidebarItem: FC<SidebarItemProps> = ({
3434
{showProgressionIcons && (
3535
<ProgressionIcon className={styles.progressionIcon} />
3636
)}
37-
<span className={styles.label}>{label}</span>
3837

39-
{/^https?:/.test(link) && <ArrowUpRightIcon className={styles.icon} />}
38+
<div className={`${styles.label} flex items-center gap-1.5`}>
39+
<span>{label}</span>
40+
41+
{/^https?:/.test(link) && <ArrowUpRightIcon className={styles.icon} />}
42+
</div>
4043
</BaseActiveLink>
4144
);
4245

0 commit comments

Comments
 (0)