Skip to content

feat(textarea): add styles for textarea solid fill#30685

Merged
JoaoFerreira-FrontEnd merged 16 commits into
nextfrom
ROU-12225
Sep 25, 2025
Merged

feat(textarea): add styles for textarea solid fill#30685
JoaoFerreira-FrontEnd merged 16 commits into
nextfrom
ROU-12225

Conversation

@JoaoFerreira-FrontEnd

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd commented Sep 18, 2025

Copy link
Copy Markdown
Contributor

Issue number: internal


What is the new behavior?

  • highlight should only appear on md;
  • fix some textarea ionic css vars usage;
  • create sheet for ionic.solid;

Does this introduce a breaking change?

  • Yes
  • No

Other information

fill
color
states

- fix some textarea ionic css vars usage;
- create sheet for ionic.solid;
@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd added the package: core @ionic/core package label Sep 18, 2025
@vercel

vercel Bot commented Sep 18, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
ionic-framework Ready Ready Preview Comment Sep 25, 2025 5:24pm

@brandyscarney brandyscarney left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great work! A few questions on some styles & a requested change for a variable that appears different in Figma. 👍

Comment thread core/src/components/textarea/textarea.ionic.scss Outdated
Comment thread core/src/components/textarea/textarea.ionic.solid.scss
Comment thread core/src/components/textarea/textarea.ionic.solid.scss Outdated
Comment thread core/src/components/textarea/textarea.ionic.solid.scss Outdated
Comment thread core/src/components/textarea/textarea.ionic.solid.scss Outdated
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
Comment thread core/src/components/textarea/textarea.ionic.solid.scss Outdated
Comment thread core/src/components/textarea/textarea.ionic.outline.scss

@brandyscarney brandyscarney left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good!

@ShaneK ShaneK left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Minor request, but not blocking. Looks good otherwise, great job!

Comment thread core/src/components/textarea/textarea.ionic.solid.scss
@JoaoFerreira-FrontEnd
JoaoFerreira-FrontEnd merged commit 42f3447 into next Sep 25, 2025
49 checks passed
@JoaoFerreira-FrontEnd
JoaoFerreira-FrontEnd deleted the ROU-12225 branch September 25, 2025 17:43
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