repo sync

components/release-notes/GHAEReleaseNotePatch.tsx

@@ -26,7 +26,7 @@ export function GHAEReleaseNotePatch({ patch, didEnterView }: Props) {
     >
       <header
         style={{ zIndex: 1 }}
-        className="container-xl position-sticky top-0 color-bg-subtle border-bottom px-3 pt-4 pb-2"
+        className="container-xl color-bg-subtle border-bottom px-3 pt-4 pb-2"
       >
         <div className="d-flex flex-items-center">
           <h2 className="border-bottom-0 m-0 p-0">{patch.title}</h2>

components/release-notes/GHAEReleaseNotes.tsx

@@ -5,6 +5,8 @@ import { GHAEReleaseNotePatch } from './GHAEReleaseNotePatch'
 import { GHAEReleaseNotesContextT } from './types'
 import { MarkdownContent } from 'components/ui/MarkdownContent'
 
+import styles from './PatchNotes.module.scss'
+
 type GitHubAEProps = {
   context: GHAEReleaseNotesContextT
 }
@@ -35,8 +37,10 @@ export function GHAEReleaseNotes({ context }: GitHubAEProps) {
       </article>
 
       <aside
-        className="position-sticky top-0 d-none d-md-block border-left no-print color-bg-default flex-shrink-0"
-        style={{ width: 260, height: '100vh' }}
+        className={cx(
+          'position-sticky d-none d-md-block border-left no-print color-bg-default flex-shrink-0',
+          styles.aside
+        )}
       >
         <nav className="height-full overflow-auto">
           <MarkdownContent data-search="article-content">

components/release-notes/GHESReleaseNotePatch.tsx

@@ -40,7 +40,7 @@ export function GHESReleaseNotePatch({
     >
       <header
         style={{ zIndex: 1 }}
-        className="container-xl position-sticky top-0 color-bg-subtle border-bottom px-3 pt-4 pb-2"
+        className="container-xl color-bg-subtle border-bottom px-3 pt-4 pb-2"
       >
         <div className="d-flex flex-items-center">
           <h2 className="border-bottom-0 m-0 p-0">

components/release-notes/GHESReleaseNotes.tsx

@@ -15,6 +15,8 @@ import { MarkdownContent } from 'components/ui/MarkdownContent'
 import { GHESReleaseNotesContextT } from './types'
 import { GHESReleaseNotePatch } from './GHESReleaseNotePatch'
 
+import styles from './PatchNotes.module.scss'
+
 type Props = {
   context: GHESReleaseNotesContextT
 }
@@ -82,8 +84,10 @@ export function GHESReleaseNotes({ context }: Props) {
       </article>
 
       <aside
-        className="position-sticky top-0 d-none d-md-block border-left no-print color-bg-default flex-shrink-0"
-        style={{ width: 260, height: '100vh' }}
+        className={cx(
+          'position-sticky d-none d-md-block border-left no-print color-bg-default flex-shrink-0',
+          styles.aside
+        )}
       >
         <nav className="height-full overflow-auto">
           <MarkdownContent data-search="article-content">

components/release-notes/PatchNotes.module.scss

@@ -1,8 +1,11 @@
 @import "@primer/css/layout/index.scss";
 
 .sectionHeading {
-  scroll-margin-top: 280px !important;
-  @include breakpoint(sm) {
-    scroll-margin-top: 200px !important;
-  }
+  scroll-margin-top: 70px !important;
+}
+
+.aside {
+  width: 260px;
+  height: calc(100vh - 70px);
+  top: 70px;
 }

lib/rest/index.js

@@ -12,9 +12,11 @@ export default async function getRest() {
   const operations = {}
   fs.readdirSync(schemasPath).forEach((filename) => {
     // In staging deploys, the `.json` files might have been converted to
-    // to `.json.br`. The `readCompressedJsonFileFallback()` function
-    // can handle both but you need to call it with the `.json` filename.
-    const key = path.parse(filename).name
+    // to `.json.br`. In that case, we need to also remove the `.json`
+    // extension from the key
+    const key = path.parse(filename).name.replace('.json', '')
+    // The `readCompressedJsonFileFallback()` function
+    // will check for both a .br and .json extension.
     const value = readCompressedJsonFileFallback(path.join(schemasPath, filename))
     operations[key] = value
   })