Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(input): guarantee ionic input label doesn't have horizontal padding so it's aligned with input outline #30152

Open
wants to merge 11 commits into
base: next
Choose a base branch
from

Conversation

JoaoFerreira-FrontEnd
Copy link

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd commented Jan 23, 2025

Issue number: internal


What is the current behavior?

There were a misalignment between the labels' input and the labels' Textarea.

What is the new behavior?

  • Updates label text padding to match textarea structure
  • Updates helper and counter text padding to match textarea structure
  • Updates e2e tests snapshots to include changes

Does this introduce a breaking change?

  • Yes
  • No

Other information

input preview
textarea preview

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd added package: core @ionic/core package type: bug a confirmed bug report labels Jan 23, 2025
@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd requested a review from a team as a code owner January 23, 2025 11:15
Copy link

vercel bot commented Jan 23, 2025

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 Jan 27, 2025 6:24pm

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd self-assigned this Jan 23, 2025
- update textarea border-color;
- update input colors;
- update input border radius;
core/src/components/input/input.ionic.scss Outdated Show resolved Hide resolved
core/src/components/input/input.ionic.scss Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

We should create a separate task to update the focus color for textarea since we are updating input.

core/src/components/textarea/textarea.ionic.scss Outdated Show resolved Hide resolved
JoaoFerreira-FrontEnd and others added 3 commits January 27, 2025 18:18
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: core @ionic/core package type: bug a confirmed bug report
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants