Skip to content

fix: mark placeholder option as selected in react SfSelect (#3263)#3265

Merged
Szymon-dziewonski merged 3 commits intov2from
v2-release/patch-select
Jan 20, 2025
Merged

fix: mark placeholder option as selected in react SfSelect (#3263)#3265
Szymon-dziewonski merged 3 commits intov2from
v2-release/patch-select

Conversation

@Szymon-dziewonski
Copy link
Contributor

Related issue

Closes #

Scope of work

Screenshots of visual changes

Checklist

  • Self code-reviewed
  • Changes documented
  • Semantic HTML
  • SSR-friendly
  • Caching friendly
  • a11y for WCAG 2.0 AA
  • examples created
  • blocks created
  • cypress tests created

* fix: mark placeholder option as selected in react SfSelect

* chore: changeset

* refactor: add internal value to vue SfSelect

* chore: changeset for vue

* refactor: remove redundant <script setup /> from vue components

* refactor: internal state in SfSelect

* refactor: remove disabled and selected attributes

* chore: remove disabled in vue placeholder option

* Revert "refactor: remove redundant <script setup /> from vue components"

This reverts commit dafb778.

* refactor: set inheritAttrs through defineOptions
@changeset-bot
Copy link

changeset-bot bot commented Jan 20, 2025

⚠️ No Changeset found

Latest commit: f2d97a3

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

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

Click here to learn what changesets are, and how to add one.

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

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@Szymon-dziewonski Szymon-dziewonski merged commit 3a2628b into v2 Jan 20, 2025
28 checks passed
@Szymon-dziewonski Szymon-dziewonski deleted the v2-release/patch-select branch January 20, 2025 12:18
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