Skip to content

Commit

Permalink
Fix an issue where the active site section indicator appeared above a…
Browse files Browse the repository at this point in the history
…ny dropdowns. (#2568)
  • Loading branch information
emmerich authored Nov 19, 2024
1 parent 7ba67fd commit ca134c8
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 5 deletions.
5 changes: 5 additions & 0 deletions .changeset/large-hornets-exist.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'gitbook': minor
---

Fix an issue where the active site section indicator appeared above any dropdowns.
1 change: 0 additions & 1 deletion packages/gitbook/src/components/Header/Dropdown.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ export function Dropdown<E extends HTMLElement>(props: {
'absolute',
'top-full',
'left-0',
'z-20',
'origin-top-left',
'invisible',
'transition-opacity',
Expand Down
4 changes: 2 additions & 2 deletions packages/gitbook/src/components/Header/Header.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -72,11 +72,11 @@ export function Header(props: {
)}
>
<HeaderLogo site={site} space={space} customization={customization} />
<span>
<div className="z-20">
{!hasSiteSections && isMultiVariants ? (
<SpacesDropdown space={space} spaces={spaces} />
) : null}
</span>
</div>
<HeaderLinks>
{customization.header.links.map((link, index) => {
return (
Expand Down
2 changes: 1 addition & 1 deletion packages/gitbook/src/components/Header/HeaderLinkMore.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ export function HeaderLinkMore(props: {
);

return (
<div className={`${styles.linkEllipsis} items-center`}>
<div className={`${styles.linkEllipsis} items-center z-20`}>
<Dropdown button={renderButton} className="-translate-x-48 md:translate-x-0">
<DropdownMenu>
{links.map((link, index) => (
Expand Down
2 changes: 1 addition & 1 deletion packages/gitbook/src/components/Header/HeaderLinks.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ export async function HeaderLinks({ children }: HeaderLinksProps) {
<div
className={tcls(
styles.containerHeaderlinks,
'flex justify-end items-center gap-x-2.5 mr-2.5 lg:gap-x-5 lg:mr-2.5 *:max-w-56',
'flex justify-end items-center gap-x-2.5 mr-2.5 lg:gap-x-5 lg:mr-2.5 *:max-w-56 z-20',
)}
>
{children}
Expand Down

0 comments on commit ca134c8

Please sign in to comment.