Skip to content

Commit 473ebe0

Browse files
committed
Fixes and tweaks
1 parent f7e2453 commit 473ebe0

File tree

5 files changed

+98
-66
lines changed

5 files changed

+98
-66
lines changed

pnpm-lock.yaml

Lines changed: 50 additions & 27 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

website/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
"dependencies": {
1818
"@docsearch/react": "^3.8.3",
1919
"@edgeandnode/common": "^6.51.0",
20-
"@edgeandnode/gds": "6.3.0-for-docs-1739342025264-8ad495310eda52f8d29773ce23b865279ba859e1",
20+
"@edgeandnode/gds": "6.3.0-for-docs-1739382074301-3bbbe172850b88e0729cded6a0f7c7c0e9b9dcd1",
2121
"@edgeandnode/go": "8.9.0-for-docs-1739342025264-8ad495310eda52f8d29773ce23b865279ba859e1",
2222
"@emotion/react": "^11.14.0",
2323
"@graphprotocol/contracts": "6.2.1",

website/src/Layout.tsx

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -337,14 +337,14 @@ export default function Layout({ pageOpts, children }: NextraThemeLayoutProps<Fr
337337
className="text-body-large flex items-center text-text outline-offset-2 transition hocus-visible:text-white md:-ms-1 md:me-auto"
338338
>
339339
<span className="absolute -inset-2" />
340-
<TheGraph size={7} color="white" />
340+
<TheGraph color="white" className="prop-size-8 md:prop-size-7" />
341341
<span className="flex items-center max-md:hidden">
342342
<span className="me-4.5 ms-3.5 h-9 w-px shrink-0 bg-white/8" />
343343
<span>Docs</span>
344344
</span>
345345
</ButtonOrLink>
346-
{/* TODO: Use naked `ExperimentalButton` after new design is implemented */}
347346
<div className="order-first px-[var(--graph-docs-header-padding)] md:order-none md:px-0">
347+
{/* TODO: Use `ExperimentalButton` after new design is implemented (tertiary on mobile, naked on desktop) */}
348348
<button
349349
type="button"
350350
data-sidebar-expanded-on-mobile={sidebarExpandedOnMobile || undefined}
@@ -424,6 +424,7 @@ export default function Layout({ pageOpts, children }: NextraThemeLayoutProps<Fr
424424
translations={translations.docsearch as NestedStrings}
425425
>
426426
{({ buttonRef, openModal }) => (
427+
// TODO: Use tertiary `ExperimentalButton` after new design is implemented, at least on mobile
427428
<button
428429
ref={buttonRef}
429430
type="button"
@@ -435,7 +436,7 @@ export default function Layout({ pageOpts, children }: NextraThemeLayoutProps<Fr
435436
lg:w-60 lg:border lg:text-white/64 lg:outline-none lg:hocus-visible:text-white/88 lg:hover:border-white/32
436437
`}
437438
>
438-
<MagnifyingGlass alt="" className="prop-size-4 lg:prop-size-3" />
439+
<MagnifyingGlass alt="" className="prop-size-5 lg:prop-size-3" />
439440
<span className="text-body-xsmall flex-1 truncate max-lg:hidden">
440441
{t('docsearch.button.buttonText')}
441442
</span>
@@ -445,7 +446,7 @@ export default function Layout({ pageOpts, children }: NextraThemeLayoutProps<Fr
445446
</button>
446447
)}
447448
</DocSearch>
448-
{/* TODO: Implement new design */}
449+
{/* TODO: Use tertiary `ExperimentalButton` in `children` after new design is implemented */}
449450
<ExperimentalLocaleSwitcher className="prop-display-format-short lg:prop-display-format-full" />
450451
{/* TODO: Fix app launcher */}
451452
{/* <ExperimentalAppLauncher /> */}
@@ -503,6 +504,7 @@ export default function Layout({ pageOpts, children }: NextraThemeLayoutProps<Fr
503504
})(groupItem),
504505
)
505506
return (
507+
// TODO: Auto-collapse other groups when opening one via the title?
506508
<NavigationGroup key={groupIndex}>
507509
{group.title !== undefined ? (
508510
<NavigationItem
@@ -530,6 +532,7 @@ export default function Layout({ pageOpts, children }: NextraThemeLayoutProps<Fr
530532
Link,
531533
blockquote: Callout,
532534
// TODO: Build and use `ExperimentalCodeInline`
535+
// TODO: Test `ExperimentalCodeInline` and `ExperimentalLink` together
533536
code: Code.Inline as any,
534537
h1: Heading.H1,
535538
h2: Heading.H2,
@@ -828,6 +831,7 @@ function MDXContent({ toc: headings, children }: ComponentPropsWithoutRef<Nextra
828831
{t('global.page.edit')}
829832
</LegacyLink>
830833
</header>
834+
{/* TODO: How do we / should we handle inline code and links in headings? */}
831835
{headings.length > 0 ? (
832836
<nav
833837
aria-label={t('global.page.tableOfContents')}

website/src/app.css

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
11
@tailwind base;
22
@tailwind components;
33
@tailwind utilities;
4+
5+
/* Work around https://github.com/algolia/docsearch/issues/2243 */
6+
.DocSearch-Container {
7+
position: fixed !important;
8+
}

0 commit comments

Comments
 (0)