Skip to content

Bug-1982588: Align breadcrumbs and replicates button in Subtests View - #1102

Draft
moijes12 wants to merge 2 commits into
mozilla:mainfrom
moijes12:bug1982588-14536369194714119665
Draft

moijes12 wants to merge 2 commits into
mozilla:mainfrom
moijes12:bug1982588-14536369194714119665

Conversation

@moijes12

@moijes12 moijes12 commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Changes Introduced

  • Breadcrumbs and ToggleReplicatesButton now share one Grid with space-between / alignItems: center in SubtestsResultsMain.tsx (same pattern as Results View).
  • Breadcrumb marginBottom: 3 was removed so they line up.
  • Button still only shows for STUDENT_T. Snapshots updated. PR title/body include the bug number and target fork main, not mozilla/*

Please review: @beatrice-acasandrei @esanuandra @kala-moz

Fixes Bug-1982588

@netlify

netlify Bot commented Sep 10, 2026

Copy link
Copy Markdown

Deploy Preview for mozilla-perfcompare ready!

Name Link
🔨 Latest commit c501abe
🔍 Latest deploy log https://app.netlify.com/projects/mozilla-perfcompare/deploys/6aa27430edafa70008b2904c
😎 Deploy Preview https://deploy-preview-1102--mozilla-perfcompare.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

<Grid container sx={titleContainerSx}>
<SubtestsBreadcrumbs view={view} />
{testVersion === STUDENT_T && (
<Grid component='h2' className={styles.replicates}>

@beatrice-acasandrei beatrice-acasandrei Sep 16, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using component='h2' makes the Grid render as an <h2> HTML element, but here it's just a layout wrapper for a button, not a section title

Suggested change
<Grid component='h2' className={styles.replicates}>
<Grid className={styles.replicates}>


const titleContainerSx = {
alignItems: 'center',
gap: '9px',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MUI spacing unit, consistent with the rest of the codebase

Suggested change
gap: '9px',
gap: 1,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants