Conversation
There was a problem hiding this comment.
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
buttonOuterprop, these buttons will no longer render a.btn-outerwrapper.ProgressBar.scssstill 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-outerrule 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.
There was a problem hiding this comment.
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.
|
@rammodhvadia looks like some valid Copilot comments. |
various fixes for projects-ui: