Merge pull request #25027 from github/repo-sync

repo sync

components/page-footer/Contribution.tsx

@@ -13,7 +13,7 @@ export const Contribution = () => {
 
   return (
     <div className="f5 contribution">
-      <h2 className="f4 mb-3">{t`title`}</h2>
+      <h3 className="f4 mb-3">{t`title`}</h3>
       <p className="max-w-xs color-fg-muted mb-3">{t`body`}</p>
       <a className="btn color-border-accent-emphasis" href={contributionHref}>
         <GitPullRequestIcon size="small" className="octicon mr-1" />

components/page-footer/LegalFooter.tsx

@@ -9,7 +9,7 @@ export const LegalFooter = () => {
   const { t } = useTranslation('footer')
   return (
     <section className="container-xl px-3 mt-6 pb-8 px-md-6 color-fg-muted">
-      <h1 className="f4 mb-2 col-12">{t('legal_heading')}</h1>
+      <h2 className="f4 mb-2 col-12">{t('legal_heading')}</h2>
       <ul className="d-flex flex-wrap list-style-none">
         <li className="mr-3">&copy; {new Date().getFullYear()} GitHub, Inc.</li>
         {/* In Germany, Austria, and Switzerland, the Impressum link is legally required. */}

components/page-footer/Support.tsx

@@ -9,7 +9,7 @@ export const Support = () => {
 
   return (
     <div>
-      <h2 className="mb-3 f4">{t`still_need_help`}</h2>
+      <h3 className="mb-3 f4">{t`still_need_help`}</h3>
       <div className="mb-2">
         <a
           id="ask-community"

components/page-footer/SupportSection.tsx

@@ -28,7 +28,7 @@ export const SupportSection = () => {
 
   return (
     <section className="container-xl mt-lg-8 mt-6 px-3 px-md-6 no-print mx-auto">
-      <h1 className="f3">{t('support_heading')}</h1>
+      <h2 className="f3">{t('support_heading')}</h2>
       <div className="container-xl mx-auto py-6 py-lg-6 clearfix border-top border-color-secondary">
         {showSurvey && (
           <div

components/page-footer/Survey.tsx

@@ -80,7 +80,7 @@ export const Survey = () => {
 
   return (
     <form className="f5" onSubmit={submit} ref={formRef} data-testid="survey-form">
-      <h2 className="f4 mb-3">{t`able_to_find`}</h2>
+      <h3 className="f4 mb-3">{t`able_to_find`}</h3>
 
       {/* Honeypot: token isn't a real field */}
       <input type="text" className="d-none" name="survey-token" aria-hidden="true" />

tests/browser/browser.js

@@ -145,22 +145,6 @@ describe('browser search', () => {
   })
 })
 
-describe('x-large viewports - 1280+', () => {
-  jest.setTimeout(60 * 1000)
-  it('in article breadcrumbs at xl viewport should remove last breadcrumb', async () => {
-    await page.setViewport({ width: 1300, height: 700 })
-    await page.goto(
-      'http://localhost:4000/en/enterprise-cloud@latest/admin/identity-and-access-management/managing-iam-for-your-enterprise/about-authentication-for-your-enterprise'
-    )
-    const breadcrumbsElement = await page.$$('[data-testid=breadcrumbs-in-article] ul li')
-    const breadcrumbsMissingElement = await page.$$(
-      '[data-testid=breadcrumbs-in-article] ul li .d-none'
-    )
-    expect(breadcrumbsMissingElement.length).toBe(1)
-    expect(breadcrumbsElement.length).toBe(4)
-  })
-})
-
 describe('large -> x-large viewports - 1012+', () => {
   jest.setTimeout(60 * 1000)
   it('version picker is visible in header', async () => {

tests/rendering-fixtures/playwright-rendering.spec.ts

@@ -228,3 +228,17 @@ test('hovercards', async ({ page }) => {
   await page.locator('#article-contents').getByRole('link', { name: 'introduction' }).hover()
   await expect(page.getByText('You can use GitHub Pages to showcase')).toBeVisible()
 })
+
+test('x-large viewports - 1280+', async ({ page }) => {
+  page.setViewportSize({
+    width: 1300,
+    height: 700,
+  })
+  await page.goto('/get-started/foo/bar')
+
+  // in article breadcrumbs at xl viewport should remove last breadcrumb so
+  // for this page we should only have 'Get Started / Foo'
+  expect(await page.getByTestId('breadcrumbs-in-article').getByRole('link').all()).toHaveLength(2)
+  await expect(page.getByTestId('breadcrumbs-in-article').getByText('Foo')).toBeVisible()
+  await expect(page.getByTestId('breadcrumbs-in-article').getByText('Bar')).not.toBeVisible()
+})