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 8 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 24, 2025 7:29pm

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd self-assigned this Jan 23, 2025
- update textarea border-color;
- update input colors;
- update input border radius;
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.

1 participant