Skip to content

AUT-5636: add property to choose primary button - #1090

Merged
hatimdalal1 merged 4 commits into
mainfrom
hdalal/AUT-5636-make-back-button-primary
Aug 14, 2026
Merged

AUT-5636: add property to choose primary button#1090
hatimdalal1 merged 4 commits into
mainfrom
hdalal/AUT-5636-make-back-button-primary

Conversation

@hatimdalal1

@hatimdalal1 hatimdalal1 commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

JIRA

Description

  • Adding property to make Back and Restart button primary
  • Purpose for this is we want one of the buttons on the left to be set as primary when the next button (on the right) is disabled

@hatimdalal1
hatimdalal1 requested a review from a team as a code owner August 14, 2026 12:47
@hatimdalal1
hatimdalal1 marked this pull request as draft August 14, 2026 12:48
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for the PR! 🎉

We've deployed an automatic preview for this PR - you can see your changes here:

URL https://live.d2l.dev/prs/BrightspaceUI/labs/pr-1090/

Note

The build needs to finish before your changes are deployed.
Changes to the PR will automatically update the instance.

@hatimdalal1
hatimdalal1 marked this pull request as ready for review August 14, 2026 13:03
Comment thread src/components/wizard/README.md Outdated
| `back-button-title` | String | Text that is displayed within the button |
| `back-button-tooltip` | String | Text that is displayed within the button tooltip |
| `display-back-button` | Boolean | Display the Back button |
| `primary-back-button` | Boolean | Make the Back button primary |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Idea: since next is primary by default and presumably design only ever wants one of them to be primary (or is Next always still primary?), you could do something like primary-button="back|restart|next".

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yes, I can do that. And I'll keep "next" as default primary to ensure backwards compatibility

@hatimdalal1
hatimdalal1 requested a review from dlockhart August 14, 2026 13:56

@dlockhart dlockhart left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good!

@hatimdalal1 hatimdalal1 changed the title AUT-5636: add property to make back and restart buttons primary AUT-5636: add property to choose primary button Aug 14, 2026
@hatimdalal1
hatimdalal1 merged commit e5444f3 into main Aug 14, 2026
8 of 9 checks passed
@hatimdalal1
hatimdalal1 deleted the hdalal/AUT-5636-make-back-button-primary branch August 14, 2026 17:36
@d2l-github-release-tokens

Copy link
Copy Markdown

🎉 This PR is included in version 2.60.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants