Skip to content

refactor(ui): Rename layout prop to options - #4209

Merged
alexcarpenter merged 4 commits into
mainfrom
alexcarpenter/ui-appearance-options
Sep 24, 2024
Merged

refactor(ui): Rename layout prop to options#4209
alexcarpenter merged 4 commits into
mainfrom
alexcarpenter/ui-appearance-options

Conversation

@alexcarpenter

@alexcarpenter alexcarpenter commented Sep 23, 2024

Copy link
Copy Markdown
Member

Description

Slack thread for more info https://clerkinc.slack.com/archives/C076A87HCM6/p1726850512663219

Updates appearance layout prop naming to be options

Checklist

  • npm test runs as expected.
  • npm run build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@changeset-bot

changeset-bot Bot commented Sep 23, 2024

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 5d4e70f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@alexcarpenter
alexcarpenter marked this pull request as ready for review September 23, 2024 22:26
@alexcarpenter

Copy link
Copy Markdown
Member Author

Will merge for now, we can always revert to layout naming if we feel strongly it should stick.

@alexcarpenter
alexcarpenter merged commit d61bef1 into main Sep 24, 2024
@alexcarpenter
alexcarpenter deleted the alexcarpenter/ui-appearance-options branch September 24, 2024 21:06
wobsoriano pushed a commit that referenced this pull request Feb 8, 2025
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