From f5df15a5c890d3ec0c598d1350cc4b4485d58b26 Mon Sep 17 00:00:00 2001 From: Kevin Heis Date: Wed, 8 Oct 2025 11:07:23 -0700 Subject: [PATCH] Replace `sx` and inline `style` declarations with CSS modules (#57848) Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- .../components/GroupedEvents.module.scss | 16 +++++++++ src/audit-logs/components/GroupedEvents.tsx | 14 ++++---- .../components/DefaultLayout.module.scss | 3 ++ src/frame/components/DefaultLayout.tsx | 6 ++-- src/frame/components/GenericError.module.scss | 3 ++ src/frame/components/GenericError.tsx | 4 ++- .../components/page-header/Header.module.scss | 4 +++ src/frame/components/page-header/Header.tsx | 9 +++-- .../HeaderSearchAndWidgets.module.scss | 26 +++++++++++++++ .../page-header/HeaderSearchAndWidgets.tsx | 32 ++---------------- .../components/sidebar/SidebarNav.module.scss | 33 +++++++++++++++++++ src/frame/components/sidebar/SidebarNav.tsx | 29 ++++++++-------- .../MarkdownContent.module.scss | 11 +++++++ .../UnrenderedMarkdownContent.tsx | 12 +++---- src/frame/components/ui/MiniTocs/MiniTocs.tsx | 8 +---- .../ui/MiniTocs/Minitocs.module.scss | 16 +++++++++ .../PermissionsStatement.module.scss | 6 ++++ .../PermissionsStatement.tsx | 7 ++-- .../ui/ScrollButton/ScrollButton.module.scss | 5 +++ .../ui/ScrollButton/ScrollButton.tsx | 3 +- src/graphql/pages/explorer.module.scss | 3 ++ src/graphql/pages/explorer.tsx | 5 +-- .../components/LanguagePicker.module.scss | 12 +++++++ src/languages/components/LanguagePicker.tsx | 11 +++---- .../components/GHESReleaseNotePatch.tsx | 8 ++--- .../components/PatchNotes.module.scss | 9 +++++ .../components/RestCodeSamples.module.scss | 1 + src/rest/components/RestCodeSamples.tsx | 25 -------------- src/rest/components/RestOperation.module.scss | 4 +++ src/rest/components/RestOperation.tsx | 6 ++-- src/tools/components/Fields.module.scss | 7 ++++ src/tools/components/Fields.tsx | 12 +++---- src/tools/components/Picker.module.scss | 12 +++++++ src/tools/components/Picker.tsx | 10 ++---- src/webhooks/components/Webhook.tsx | 4 +-- .../WebhookPayloadExample.module.scss | 4 +++ 36 files changed, 246 insertions(+), 134 deletions(-) create mode 100644 src/audit-logs/components/GroupedEvents.module.scss create mode 100644 src/frame/components/DefaultLayout.module.scss create mode 100644 src/frame/components/GenericError.module.scss create mode 100644 src/frame/components/sidebar/SidebarNav.module.scss create mode 100644 src/frame/components/ui/PermissionsStatement/PermissionsStatement.module.scss create mode 100644 src/graphql/pages/explorer.module.scss create mode 100644 src/languages/components/LanguagePicker.module.scss create mode 100644 src/tools/components/Picker.module.scss diff --git a/src/audit-logs/components/GroupedEvents.module.scss b/src/audit-logs/components/GroupedEvents.module.scss new file mode 100644 index 0000000000..b0c6f70f3e --- /dev/null +++ b/src/audit-logs/components/GroupedEvents.module.scss @@ -0,0 +1,16 @@ +.eventItem { + margin-bottom: 3rem; +} + +.eventAction { + font-style: normal; +} + +.eventDetail { + margin-left: 1rem; + font-style: normal; +} + +.eventDescription { + margin-left: 1rem; +} diff --git a/src/audit-logs/components/GroupedEvents.tsx b/src/audit-logs/components/GroupedEvents.tsx index d5c161b03f..1240f328bc 100644 --- a/src/audit-logs/components/GroupedEvents.tsx +++ b/src/audit-logs/components/GroupedEvents.tsx @@ -4,6 +4,8 @@ import { HeadingLink } from '@/frame/components/article/HeadingLink' import { useTranslation } from '@/languages/components/useTranslation' import type { AuditLogEventT } from '../types' +import styles from './GroupedEvents.module.scss' + type Props = { auditLogEvents: AuditLogEventT[] category: string @@ -47,15 +49,15 @@ export default function GroupedEvents({ auditLogEvents, category, categoryNote } )}
{auditLogEvents.map((event) => ( -
+
-
+
{event.action}
{event.description}
-
{t('fields')}
-
+
{t('fields')}
+
{event.fields ? event.fields.map((field, index) => ( @@ -68,8 +70,8 @@ export default function GroupedEvents({ auditLogEvents, category, categoryNote } {event.docs_reference_links && event.docs_reference_links !== 'N/A' && ( <> -
{t('reference')}
-
{renderReferenceLinks(event)}
+
{t('reference')}
+
{renderReferenceLinks(event)}
)}
diff --git a/src/frame/components/DefaultLayout.module.scss b/src/frame/components/DefaultLayout.module.scss new file mode 100644 index 0000000000..3612da8c34 --- /dev/null +++ b/src/frame/components/DefaultLayout.module.scss @@ -0,0 +1,3 @@ +.mainContent { + scroll-margin-top: 5rem; +} diff --git a/src/frame/components/DefaultLayout.tsx b/src/frame/components/DefaultLayout.tsx index 69f8db0963..5c3d5bf299 100644 --- a/src/frame/components/DefaultLayout.tsx +++ b/src/frame/components/DefaultLayout.tsx @@ -15,6 +15,8 @@ import { useLanguages } from '@/languages/components/LanguagesContext' import { ClientSideLanguageRedirect } from './ClientSideLanguageRedirect' import { SearchOverlayContextProvider } from '@/search/components/context/SearchOverlayContext' +import styles from './DefaultLayout.module.scss' + const MINIMAL_RENDER = Boolean(JSON.parse(process.env.MINIMAL_RENDER || 'false')) type Props = { children?: React.ReactNode } @@ -51,7 +53,7 @@ export const DefaultLayout = (props: Props) => {
-
+
{props.children}
@@ -150,7 +152,7 @@ export const DefaultLayout = (props: Props) => { {/* Need to set an explicit height for sticky elements since we also set overflow to auto */}
-
+
diff --git a/src/frame/components/GenericError.module.scss b/src/frame/components/GenericError.module.scss new file mode 100644 index 0000000000..247697d1cb --- /dev/null +++ b/src/frame/components/GenericError.module.scss @@ -0,0 +1,3 @@ +.logoContainer { + z-index: 3; +} diff --git a/src/frame/components/GenericError.tsx b/src/frame/components/GenericError.tsx index 3e5f0b36c7..d2b59d47de 100644 --- a/src/frame/components/GenericError.tsx +++ b/src/frame/components/GenericError.tsx @@ -5,6 +5,8 @@ import { useRouter } from 'next/router' import { MarkGithubIcon, CommentDiscussionIcon } from '@primer/octicons-react' import { Lead } from '@/frame/components/ui/Lead' +import styles from './GenericError.module.scss' + export function GenericError() { return (
@@ -44,7 +46,7 @@ export const SimpleHeader = () => { role="banner" aria-label="Main" > -
+
diff --git a/src/frame/components/page-header/Header.module.scss b/src/frame/components/page-header/Header.module.scss index 27b117d456..3a5e1285e4 100644 --- a/src/frame/components/page-header/Header.module.scss +++ b/src/frame/components/page-header/Header.module.scss @@ -57,3 +57,7 @@ padding: 0 !important; } } + +.headerContainer { + row-gap: 1rem; +} diff --git a/src/frame/components/page-header/Header.tsx b/src/frame/components/page-header/Header.tsx index ed41545670..07e8409fcb 100644 --- a/src/frame/components/page-header/Header.tsx +++ b/src/frame/components/page-header/Header.tsx @@ -156,11 +156,10 @@ export const Header = () => { aria-label="Main" >