Skip to content

Commit

Permalink
Use consistent link color
Browse files Browse the repository at this point in the history
  • Loading branch information
damianstasik committed Sep 19, 2023
1 parent 456a87e commit 0bbc50f
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 8 deletions.
7 changes: 5 additions & 2 deletions src/components/SocialIconLink/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ export default function SocialIconLink({
return (
<Link
href={href}
className="flex items-center gap-3 text-gray-900 dark:text-gray-50 hover:text-gray-500 hover:dark:text-gray-500"
className="flex items-center gap-3 text-gray-900 hover:text-brand-700 dark:text-gray-50 dark:hover:text-brand-500"
aria-label={label}
>
<svg
Expand All @@ -38,7 +38,10 @@ export default function SocialIconLink({
>
<path fill="currentColor" d={iconMap[name]} />
</svg>
<span className={clsx("inline-flex xl:hidden", hiddenLabel && "hidden")} aria-hidden>
<span
className={clsx("inline-flex xl:hidden", hiddenLabel && "hidden")}
aria-hidden
>
{label}
</span>
</Link>
Expand Down
2 changes: 1 addition & 1 deletion src/theme/Logo/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ export default function Logo() {
return (
<Link
to={logoLink}
className="text-gray-900 dark:text-gray-50"
className="text-gray-900 hover:text-brand-700 dark:text-gray-50 dark:hover:text-brand-500"
aria-label="Go to homepage"
>
<LogoSvg aria-hidden className="h-12 mb-3" />
Expand Down
2 changes: 1 addition & 1 deletion src/theme/Navbar/MobileSidebar/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ function CloseButton() {
<button
type="button"
aria-label="Close navigation bar"
className="absolute top-6 right-6 p-4 text-gray-900 dark:text-gray-50"
className="absolute top-6 right-6 p-4 text-gray-900 hover:text-brand-700 dark:text-gray-50 dark:hover:text-brand-500 transition-colors"
onClick={() => mobileSidebar.toggle()}
>
<IconClose />
Expand Down
4 changes: 2 additions & 2 deletions src/theme/NavbarItem/DefaultNavbarItem.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ function DefaultNavbarItemDesktop({
const element = (
<NavbarNavLink
className={clsx(
"font-bold text-gray-900 dark:text-gray-50 hover:text-gray-900 hover:dark:text-gray-50",
"font-bold text-gray-900 dark:text-gray-50 hover:text-brand-700 dark:hover:text-brand-500",
className
)}
isDropdownLink={isDropdownItem}
Expand All @@ -33,7 +33,7 @@ function DefaultNavbarItemMobile({
return (
<NavbarNavLink
className={clsx(
"font-bold text-gray-900 dark:text-gray-50 hover:text-gray-900 hover:dark:text-gray-50",
"font-bold text-gray-900 dark:text-gray-50 hover:text-gray-900 dark:hover:text-brand-500",
className
)}
{...props}
Expand Down
7 changes: 5 additions & 2 deletions src/theme/NavbarItem/DropdownNavbarItem.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ function DropdownNavbarItemDesktop({
role="button"
href={props.to ? undefined : "#"}
className={clsx(
"peer flex items-center font-bold text-gray-900 dark:text-gray-50 group-hover:text-gray-900 group-hover:dark:text-gray-50 group-hover:underline",
"peer flex items-center font-bold text-gray-900 dark:text-gray-50 group-hover:text-brand-700 dark:group-hover:text-brand-500 group-hover:underline",
className
)}
{...props}
Expand Down Expand Up @@ -82,7 +82,10 @@ function DropdownNavbarItemMobile({
<>
<NavbarNavLink
role="button"
className={clsx("font-bold text-gray-900 dark:text-gray-50", className)}
className={clsx(
"font-bold text-gray-900 hover:text-brand-700 dark:text-gray-50 dark:hover:text-brand-500",
className
)}
{...props}
>
{props.children ?? props.label}
Expand Down

0 comments on commit 0bbc50f

Please sign in to comment.