Skip to content

Comments

Projects site styling fixes#1332

Merged
rammodhvadia merged 9 commits intomainfrom
projects-site-styling-fixes
Feb 23, 2026
Merged

Projects site styling fixes#1332
rammodhvadia merged 9 commits intomainfrom
projects-site-styling-fixes

Conversation

@rammodhvadia
Copy link
Contributor

@rammodhvadia rammodhvadia commented Feb 17, 2026

various fixes for projects-ui:

  • uses custom border radius for secondary buttons
  • improved padding around progress bar
  • fix some border spacing issues in sidebar
  • add custom padding to instruction panel

@rammodhvadia rammodhvadia temporarily deployed to previews/1332/merge February 17, 2026 15:49 — with GitHub Actions Inactive
@rammodhvadia rammodhvadia marked this pull request as ready for review February 19, 2026 14:40
Copilot AI review requested due to automatic review settings February 19, 2026 14:40
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR makes a set of styling adjustments to the projects sidebar/instructions UI, focusing on button rounding, spacing around the progress bar, and sidebar/instructions panel layout flexibility via CSS custom properties.

Changes:

  • Updates secondary button styling to support a configurable border radius.
  • Adjusts progress bar layout spacing (padding and internal spacing).
  • Adds configurable sidebar alignment/footer padding and instructions content bottom padding.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/components/Menus/Sidebar/InstructionsPanel/ProgressBar/ProgressBar.jsx Removes buttonOuter usage for step navigation buttons, affecting rendered DOM structure/styling hooks.
src/assets/stylesheets/Sidebar.scss Adds CSS variable overrides for alignment/footer padding and tweaks border obscurer positioning.
src/assets/stylesheets/ProgressBar.scss Adds vertical padding to the progress bar and introduces a small internal gap in the progress container.
src/assets/stylesheets/Instructions.scss Adds bottom padding hook for instruction content via a CSS custom property.
src/assets/stylesheets/Button.scss Adds a configurable border-radius for secondary buttons via a CSS custom property.
Comments suppressed due to low confidence (1)

src/components/Menus/Sidebar/InstructionsPanel/ProgressBar/ProgressBar.jsx:43

  • After removing the buttonOuter prop, these buttons will no longer render a .btn-outer wrapper. ProgressBar.scss still contains .progress-bar .btn-outer { ... } overrides, so verify the desired padding/click-target styling still applies and consider removing or updating the now-unused .btn-outer rule to avoid dead CSS.
        <Button
          className={"btn--secondary btn--small"}
          onClickHandler={goToPreviousStep}
          ButtonIcon={ChevronLeft}
          disabled={currentStepPosition === 0}
          title={t("instructionsPanel.previousStep")}
        />

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@adrian-rpf
Copy link
Contributor

@rammodhvadia looks like some valid Copilot comments.
Please investigate and resolve and then move this back into ready to ready

@adrian-rpf adrian-rpf marked this pull request as draft February 20, 2026 17:24
@rammodhvadia rammodhvadia temporarily deployed to previews/1332/merge February 23, 2026 08:28 — with GitHub Actions Inactive
@rammodhvadia rammodhvadia marked this pull request as ready for review February 23, 2026 08:40
@rammodhvadia rammodhvadia merged commit 72ce685 into main Feb 23, 2026
7 checks passed
@rammodhvadia rammodhvadia deleted the projects-site-styling-fixes branch February 23, 2026 15:02
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.

3 participants