Skip to content

feat(textarea): support notch with label slot#27660

Merged
liamdebeasi merged 4 commits intoFW-4409from
4409-notch
Jun 20, 2023
Merged

feat(textarea): support notch with label slot#27660
liamdebeasi merged 4 commits intoFW-4409from
4409-notch

Conversation

@liamdebeasi
Copy link
Contributor

Issue number: N/A


What is the current behavior?

Textarea support slotted labels, but the notch cut out behavior does not work.

What is the new behavior?

  • Integrates textarea with the notchController
  • Adds tests

Note: Dynamically updating the slotted content will not re-render the notch. This is being handled in a separate PR.

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 4409 notch feat(textarea): support notch with label slot 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:09
@liamdebeasi liamdebeasi requested review from a team, brandyscarney and sean-perkins and removed request for a team and sean-perkins June 15, 2023 19:09
@liamdebeasi liamdebeasi requested review from averyjohnston and removed request for brandyscarney June 20, 2023 15:14
@liamdebeasi liamdebeasi merged commit 884fecd into FW-4409 Jun 20, 2023
@liamdebeasi liamdebeasi deleted the 4409-notch branch June 20, 2023 19:48
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.

4 participants