Skip to content

Commit 5bfbec6

Browse files
authored
made changes to the nav dropdown style to maintain the centre lining and the differentiator in the option avaialable with dropdown (calcom#27038)
1 parent ddcac4b commit 5bfbec6

1 file changed

Lines changed: 20 additions & 11 deletions

File tree

apps/web/modules/shell/navigation/NavigationItem.tsx

Lines changed: 20 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -177,23 +177,32 @@ export const NavigationItem: React.FC<{
177177
}
178178
}}
179179
className={classNames(
180-
"todesktop:py-[7px] text-default group flex w-full items-center rounded-md px-2 py-1.5 text-sm font-medium transition",
180+
"todesktop:py-[7px] text-default group relative flex w-full items-center rounded-md px-2 py-1.5 text-sm font-medium transition",
181181
"aria-[aria-current='page']:bg-transparent!",
182182
"[&[aria-current='page']]:text-emphasis mt-0.5 text-sm",
183+
"md:justify-center lg:justify-start",
183184
isLocaleReady
184185
? "hover:bg-subtle todesktop:[&[aria-current='page']]:bg-emphasis todesktop:hover:bg-transparent hover:text-emphasis"
185186
: ""
186187
)}
187188
>
188189
{item.icon && (
189-
<Icon
190-
name={item.isLoading ? "rotate-cw" : item.icon}
191-
className={classNames(
192-
"todesktop:!text-blue-500 mr-2 h-4 w-4 shrink-0 rtl:ml-2 md:ltr:mx-auto lg:ltr:mr-2",
193-
item.isLoading && "animate-spin"
190+
<div className="relative">
191+
<Icon
192+
name={item.isLoading ? "rotate-cw" : item.icon}
193+
className={classNames(
194+
"todesktop:!text-blue-500 h-4 w-4 shrink-0 lg:ltr:mr-2 lg:rtl:ml-2",
195+
item.isLoading && "animate-spin"
196+
)}
197+
aria-hidden="true"
198+
/>
199+
{shouldShowChevron && (
200+
<Icon
201+
name={isExpanded ? "chevron-up" : "chevron-down"}
202+
className="absolute -bottom-0.5 -right-0.5 h-2.5 w-2.5 rounded-full bg-subtle p-0.5 lg:hidden"
203+
/>
194204
)}
195-
aria-hidden="true"
196-
/>
205+
</div>
197206
)}
198207
{isLocaleReady ? (
199208
<span
@@ -209,7 +218,7 @@ export const NavigationItem: React.FC<{
209218
{shouldShowChevron && (
210219
<Icon
211220
name={isExpanded ? "chevron-up" : "chevron-down"}
212-
className="ml-auto h-4 w-4"
221+
className="ml-auto hidden h-4 w-4 lg:block"
213222
/>
214223
)}
215224
</button>
@@ -233,7 +242,7 @@ export const NavigationItem: React.FC<{
233242
? "mt-0"
234243
: "mt-1 hover:mt-1 [&[aria-current='page']]:mt-1"
235244
}`
236-
: "[&[aria-current='page']]:text-emphasis mt-0.5 text-sm",
245+
: "[&[aria-current='page']]:text-emphasis mt-0.5 text-sm md:justify-center lg:justify-start",
237246
isLocaleReady
238247
? "hover:bg-subtle todesktop:[&[aria-current='page']]:bg-emphasis todesktop:hover:bg-transparent hover:text-emphasis"
239248
: ""
@@ -244,7 +253,7 @@ export const NavigationItem: React.FC<{
244253
<Icon
245254
name={item.isLoading ? "rotate-cw" : item.icon}
246255
className={classNames(
247-
"todesktop:!text-blue-500 mr-2 h-4 w-4 shrink-0 aria-[aria-current='page']:text-inherit rtl:ml-2 md:ltr:mx-auto lg:ltr:mr-2",
256+
"todesktop:!text-blue-500 h-4 w-4 shrink-0 aria-[aria-current='page']:text-inherit lg:ltr:mr-2 lg:rtl:ml-2",
248257
item.isLoading && "animate-spin"
249258
)}
250259
aria-hidden="true"

0 commit comments

Comments
 (0)