Skip to content

feat(textarea): add workaround for dynamic slot content#27661

Merged
liamdebeasi merged 17 commits intoFW-4409from
4409-dynamic
Jun 20, 2023
Merged

feat(textarea): add workaround for dynamic slot content#27661
liamdebeasi merged 17 commits intoFW-4409from
4409-dynamic

Conversation

@liamdebeasi
Copy link
Contributor

Issue number: N/A


What is the current behavior?

Slotted content added async does not cause the component to re-render.

What is the new behavior?

  • Adding slotted content async causes the component to re-render
  • Changes within the slotted content will also cause the component to re-render so the notch is updated

Does this introduce a breaking change?

  • Yes
  • No

Other information

@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@liamdebeasi liamdebeasi changed the title feat(input): add workaround for dynamic slot content feat(textarea): add workaround for dynamic slot content Jun 15, 2023
@github-actions github-actions bot added the package: core @ionic/core package label Jun 15, 2023
@liamdebeasi liamdebeasi marked this pull request as ready for review June 15, 2023 19:39
@liamdebeasi liamdebeasi requested review from a team and averyjohnston and removed request for a team June 15, 2023 19:57
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like the branch might need a rebase? I'm guessing these screenshots shouldn't be in here.

@liamdebeasi
Copy link
Contributor Author

Grabbed latest changes from feature-7.1, so both issues should be resolved.

Base automatically changed from 4409-notch to FW-4409 June 20, 2023 19:48
@liamdebeasi liamdebeasi merged commit cb31b86 into FW-4409 Jun 20, 2023
@liamdebeasi liamdebeasi deleted the 4409-dynamic branch June 20, 2023 19:51
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.

3 participants