Skip to content

Remove top-nav wordmark and mobile menu Alpha badge #97

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
feynon wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Remove top-nav wordmark and mobile menu Alpha badge #97

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 1 addition & 17 deletions components/mobile-menu.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,6 @@ export function MobileMenu({ isOpen, onClose, themeAware = false, hasBanner = fa
const headerCtaPalette = themeAware
? themeAwareHeaderPrimaryCtaClasses
: `${buttonBg} ${buttonText} ${buttonHover}`
const alphaBadgeClass = themeAware
? 'border border-black/10 bg-black/[0.04] text-black/65 dark:border-white/10 dark:bg-white/[0.06] dark:text-white/70'
: 'border border-black/10 bg-black/[0.04] text-black/65'
const headerCtaLabelClass = 'transition-opacity duration-200 group-hover:opacity-90'
const headerCtaIconClass = 'flex h-3 w-3 items-center justify-center text-[0.95em] font-medium leading-none transition-opacity duration-200 group-hover:opacity-85 sm:h-4 sm:w-4'
const mobileHeaderControlSize = 'h-7 sm:h-8.5'
Expand Down Expand Up @@ -135,7 +132,7 @@ export function MobileMenu({ isOpen, onClose, themeAware = false, hasBanner = fa
<Link
href="/"
onClick={onClose}
className="flex items-center gap-2.5 transition-opacity hover:opacity-75 sm:gap-3"
className="flex items-center transition-opacity hover:opacity-75"
>
{themeAware ? (
<>
Expand All @@ -147,19 +144,6 @@ export function MobileMenu({ isOpen, onClose, themeAware = false, hasBanner = fa
) : (
<Image src="/lighticon.png" alt="Tiles" width={56} height={56} className="h-9 w-9 sm:h-10 sm:w-10" />
)}
<span
className={`notranslate text-[1.38rem] font-semibold leading-none tracking-[-0.045em] sm:text-[1.52rem] ${textColor}`}
translate="no"
lang="en"
aria-label="Tiles"
>
Tiles
</span>
<span
className={`inline-flex items-center rounded-full px-2 py-0.5 text-[0.58rem] font-medium uppercase tracking-[0.14em] sm:text-[0.62rem] ${alphaBadgeClass}`}
>
Alpha
</span>
</Link>

{/* Right side: Download CTA and close button */}
Expand Down
12 changes: 2 additions & 10 deletions components/site-header.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -96,12 +96,12 @@ const SiteHeaderChrome = memo(function SiteHeaderChrome({
: "pt-[calc(0.875rem+env(safe-area-inset-top,0px))] sm:pt-[calc(1rem+env(safe-area-inset-top,0px))] lg:pt-[calc(1.25rem+env(safe-area-inset-top,0px))]"
} ${headerChrome} relative`}
>
{/* Left side: Logo and Wordmark */}
{/* Left side: Logo */}
<div className="flex items-center shrink-0">
<Link
href="/"
onClick={onHomeClick}
className="flex items-center gap-2.5 transition-opacity hover:opacity-75 sm:gap-3"
className="flex items-center transition-opacity hover:opacity-75"
>
{themeAware ? (
<>
Expand All @@ -113,14 +113,6 @@ const SiteHeaderChrome = memo(function SiteHeaderChrome({
) : (
<Image src="/lighticon.png" alt="Tiles" width={56} height={56} className="h-9 w-9 sm:h-10 sm:w-10 lg:h-11 lg:w-11" />
)}
<span
className={`notranslate text-[1.38rem] font-semibold leading-none tracking-[-0.028em] sm:text-[1.52rem] lg:text-[1.9rem] ${textColor}`}
translate="no"
lang="en"
aria-label="Tiles"
>
Tiles
</span>
</Link>
</div>

Expand Down