Skip to content

fix(angular): persist select disabled state in item - #29448

Merged
sean-perkins merged 2 commits into
mainfrom
sp/select-disabled-state
May 3, 2024
Merged

fix(angular): persist select disabled state in item#29448
sean-perkins merged 2 commits into
mainfrom
sp/select-disabled-state

Conversation

@sean-perkins

Copy link
Copy Markdown
Contributor

Issue number: resolves #29234


What is the current behavior?

In certain scenarios, such as in Angular where the property binding is set a few frames after the element is rendered, the ionStyle event from ion-select can be emitted before ion-item has registered an event listener.

This results in situations like setting the ion-select as initially disabled can cause the item to not treat the element as not interactable (receives pointer events).

What is the new behavior?

  • Emits the ionStyle event when the ion-select is rendered.
  • ion-item consistently detects the state of ion-select and applies the appropriate styles

Does this introduce a breaking change?

  • Yes
  • No

Other information

Forked reproduction and dev-build available here: #29234 (comment)

@vercel

vercel Bot commented May 2, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ionic-framework ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 3, 2024 3:09pm

@github-actions github-actions Bot added the package: core @ionic/core package label May 2, 2024
@sean-perkins sean-perkins changed the title fix(angular): persist select disabled state in item with IonicModule fix(angular): persist select disabled state in item May 3, 2024
@sean-perkins
sean-perkins marked this pull request as ready for review May 3, 2024 15:39
@sean-perkins
sean-perkins requested a review from a team as a code owner May 3, 2024 15:39
@sean-perkins
sean-perkins requested a review from brandyscarney May 3, 2024 15:39

@thetaPC thetaPC left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, works great

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

Labels

package: core @ionic/core package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: disabled select inside item is not being fully disabled

2 participants