Skip to content

fix: [InputText] extend clickable area (ENG-46736)#779

Open
isaquebock wants to merge 6 commits into
mainfrom
fix/input-text-refinements-eng-46736
Open

fix: [InputText] extend clickable area (ENG-46736)#779
isaquebock wants to merge 6 commits into
mainfrom
fix/input-text-refinements-eng-46736

Conversation

@isaquebock

@isaquebock isaquebock commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix para ENG-46736 — "Clique só ativa na linha do placeholder — usar base do FieldTextArea".

Aplicado h-full self-stretch no <input> interno para que ocupe toda a altura do wrapper — mesmo padrão do FieldTextArea.

How to test (mapeado 1:1 com ENG-46736)

Problema Jira: "Ao clicar em InputsText, o clique só ativa/foca quando é feito exatamente na linha do placeholder — a área clicável não cobre o campo inteiro."

  • Storybook → Components/Inputs/InputTextDefault.
  • Clique na borda superior do campo (área de padding, acima da linha do placeholder) → o input deve focar.
  • Clique na borda inferior do campo (área de padding, abaixo da linha do placeholder) → o input deve focar.
  • Repetir nos três sizes (small, medium, large) — área clicável cobre o campo inteiro em cada size.
  • Verificar que não houve regressão vs. base do FieldTextArea (o comportamento agora é equivalente).
  • Tab in/out — focus ring do input não muda.

Add h-full + self-stretch to the inner <input> so it fills the wrapper
vertically. Before this fix the input was auto-sized while the <span>
wrapper set the row height, leaving a dead band above and below the
placeholder line where clicks hit the wrapper without focusing the field.

Refs: ENG-46736
@isaquebock
isaquebock marked this pull request as ready for review July 23, 2026 17:33
@isaquebock
isaquebock requested a review from a team as a code owner July 23, 2026 17:33
@gdsantana

Copy link
Copy Markdown
Contributor
image As areas de padding laterais não estao dando focus quando o click acontece sobre elas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants