Skip to content

Commit

Permalink
Restructure NavBar (#950)
Browse files Browse the repository at this point in the history
  • Loading branch information
@majakomel
majakomel authored Sep 10, 2024
1 parent 788b4bd commit 37a57c4
Show file tree
Hide file tree
Showing 5 changed files with 147 additions and 82 deletions.
217 changes: 139 additions & 78 deletions components/NavBar.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,75 @@ const NavItem = ({ label, href, ...props }) => {
)
}

const SubNavItem = ({ label, href, ...props }) => {
const { pathname } = useRouter()
const [isActive, setIsActive] = useState(false)

useEffect(() => {
setIsActive(pathname === href)
}, [pathname, href])

return (
<Link
href={href}
className={`${isActive && 'border-black'} text-black hover:border-black hover:text-gray7 whitespace-nowrap border-b-2 py-4 px-1 text-sm font-medium`}
{...props}
>
{label}
</Link>
)
}

const SubMenu = () => {
const { pathname } = useRouter()
const [showSubMenu, setshowSubMenu] = useState(false)

const menuItem = [
{
label: <FormattedMessage id="Navbar.Charts.Circumvention" />,
href: '/chart/circumvention',
umami: 'navigation-circumvention',
},
{
label: <FormattedMessage id="Navbar.Domains" />,
href: '/domains',
umami: 'navigation-domains"',
},
{
label: <FormattedMessage id="Navbar.Networks" />,
href: '/networks',
umami: 'navigation-networks',
},
]

useEffect(() => {
setshowSubMenu(menuItem.map((i) => i.href).includes(pathname))
}, [pathname])

return (
<>
{showSubMenu && (
<div className="bg-gray-50">
<div className="border-b border-gray-200">
<div className="container">
<nav className="-mb-px flex space-x-8">
{menuItem.map((item) => (
<SubNavItem
key={item.href}
href={item.href}
label={item.label}
data-umami-event={item.umami}
/>
))}
</nav>
</div>
</div>
</div>
)}
</>
)
}

const languages = process.env.LOCALES

export const NavBar = ({ color }) => {
Expand All @@ -84,93 +153,85 @@ export const NavBar = ({ color }) => {
useEffect(() => {
setShowMenu(false)
}, [pathname])

return (
<div className={`${color} z-[8888] py-4`}>
<div className="container">
<div className="flex flex-row justify-between items-end">
<div className="z-[1] mb-1">
<Link href="/">
<ExplorerLogo height="26px" />
</Link>
</div>
<div className="StyledResponsiveMenu">
<MdMenu
size="28px"
className="lg:hidden cursor-pointer text-white"
onClick={() => setShowMenu(!showMenu)}
/>
<div
className={`z-[9999] lg:block ${showMenu ? 'block overflow-y-scroll max-h-full p-8 text-base fixed top-0 right-0 bg-gray-50' : 'hidden'}`}
>
{showMenu && (
<div className="flex justify-end">
<MdClose
size="28px"
className="cursor-pointer"
onClick={() => setShowMenu(!showMenu)}
/>
</div>
)}
<div className="z-[8888]">
<div className={`${color} py-4`}>
<div className="container">
<div className="flex flex-row justify-between items-end">
<div className="z-[1] mb-1">
<Link href="/">
<ExplorerLogo height="26px" />
</Link>
</div>
<div className="StyledResponsiveMenu">
<MdMenu
size="28px"
className="lg:hidden cursor-pointer text-white"
onClick={() => setShowMenu(!showMenu)}
/>
<div
className={`flex gap-4 lg:gap-8 ${showMenu && 'pt-2 flex-col items-start [&>a]:border-black [&>a]:hover:border-black [&>*]:opacity-100 [&>*]:text-black [&>*]:hover:text-black'}`}
className={`z-[9999] lg:block ${showMenu ? 'block overflow-y-scroll max-h-full p-8 text-base fixed top-0 right-0 bg-gray-50' : 'hidden'}`}
>
<NavItem
label={<FormattedMessage id="Navbar.Search" />}
href="/search"
data-umami-event="navigation-search"
/>
<NavItem
label={<FormattedMessage id="Navbar.Charts.MAT" />}
href="/chart/mat"
data-umami-event="navigation-mat"
/>
<NavItem
label={<FormattedMessage id="Navbar.Charts.Circumvention" />}
href="/chart/circumvention"
data-umami-event="navigation-circumvention"
/>
<NavItem
label={<FormattedMessage id="Navbar.Countries" />}
href="/countries"
data-umami-event="navigation-countries"
/>
<NavItem
label={<FormattedMessage id="Navbar.Networks" />}
href="/networks"
data-umami-event="navigation-networks"
/>
<NavItem
label={<FormattedMessage id="Navbar.Domains" />}
href="/domains"
data-umami-event="navigation-domains"
/>
<NavItem
label={<FormattedMessage id="Navbar.Findings" />}
href="/findings"
data-umami-event="navigation-findings"
/>
{user?.logged_in && (
<StyledNavItem href="" onClick={logoutUser}>
<FormattedMessage id="General.Logout" />
</StyledNavItem>
{showMenu && (
<div className="flex justify-end">
<MdClose
size="28px"
className="cursor-pointer"
onClick={() => setShowMenu(!showMenu)}
/>
</div>
)}
<LanguageSelect onChange={handleLocaleChange} value={locale}>
{languages.map((c) => (
<option
className="text-inherit opacity-100"
key={c}
value={c}
>
{getLocalisedLanguageName(c, c)}
</option>
))}
</LanguageSelect>
<div
className={`flex gap-4 lg:gap-8 ${showMenu && 'pt-2 flex-col items-start [&>a]:border-black [&>a]:hover:border-black [&>*]:opacity-100 [&>*]:text-black [&>*]:hover:text-black'}`}
>
<NavItem
label={<FormattedMessage id="Navbar.Findings" />}
href="/findings"
data-umami-event="navigation-findings"
/>
<NavItem
label={<FormattedMessage id="Navbar.Censorship" />}
href="/chart/circumvention"
data-umami-event="navigation-censorship"
/>
<NavItem
label={<FormattedMessage id="Navbar.Countries" />}
href="/countries"
data-umami-event="navigation-countries"
/>
<NavItem
label={<FormattedMessage id="Navbar.Charts" />}
href="/chart/mat"
data-umami-event="navigation-mat"
/>
<NavItem
label={<FormattedMessage id="Navbar.Search" />}
href="/search"
data-umami-event="navigation-search"
/>
{user?.logged_in && (
<StyledNavItem href="" onClick={logoutUser}>
<FormattedMessage id="General.Logout" />
</StyledNavItem>
)}
<LanguageSelect onChange={handleLocaleChange} value={locale}>
{languages.map((c) => (
<option
className="text-inherit opacity-100"
key={c}
value={c}
>
{getLocalisedLanguageName(c, c)}
</option>
))}
</LanguageSelect>
</div>
</div>
</div>
</div>
</div>
</div>
<SubMenu />
</div>
)
}
Expand Down
8 changes: 4 additions & 4 deletions components/SharedStyledComponents.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,16 +9,16 @@ export const StyledStickyNavBar = ({ ...props }) => (
<div className="sticky top-0 z-[100]" {...props} />
)

export const StyledStickySubMenu = ({ ...props }) => (
export const StyledStickySubMenu = ({ topClass, ...props }) => (
<div
className="sticky top-[62px] bg-white z-[99] border-b border-gray-300 md:mt-8 mb-2 pb-2 md:pb-0"
className={`${topClass ? topClass : 'top-[62px]'} sticky bg-white z-[99] border-b border-gray-300 md:mt-8 mb-2 pb-2 md:pb-0`}
{...props}
/>
)

export const StickySubMenu = ({ title, children }) => {
export const StickySubMenu = ({ title, children, topClass }) => {
return (
<StyledStickySubMenu>
<StyledStickySubMenu topClass={topClass}>
<div className="flex justify-between items-start md:items-center flex-col md:flex-row">
<h1 className="mt-1">{title}</h1>
{children}
Expand Down
1 change: 1 addition & 0 deletions pages/domains.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,7 @@ const Domains = () => {
</Head>
<div className="container">
<StickySubMenu
topClass="top-[116px]"
title={
<>
{intl.formatMessage({ id: 'Domains.Title' })}{' '}
Expand Down
1 change: 1 addition & 0 deletions pages/networks.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,7 @@ const Networks = () => {
</Head>
<div className="container">
<StickySubMenu
topClass="top-[116px]"
title={
<>
{intl.formatMessage({ id: 'Networks.Title' })}{' '}
Expand Down
2 changes: 2 additions & 0 deletions public/static/lang/en.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -274,9 +274,11 @@
"Measurement.Feedback.Login.Description": "We will send a link to your email. We do not store emails.",
"Measurement.Feedback.ExistingFeedback": "Your previous feedback",
"Navbar.Search": "Search",
"Navbar.Censorship": "Censorship",
"Navbar.Countries": "Countries",
"Navbar.Charts.Circumvention": "Circumvention Charts",
"Navbar.Charts.MAT": "MAT Charts",
"Navbar.Charts": "Charts",
"Navbar.Domains": "Domains",
"Navbar.Networks": "Networks",
"Navbar.Findings": "Findings",
Expand Down

0 comments on commit 37a57c4

Please sign in to comment.