Skip to content

feat(input): improve autofill - #17840

Merged
simeonoff merged 7 commits into
masterfrom
apetrov/fix-input-autofill
Oct 28, 2026
Merged

simeonoff merged 7 commits into
masterfrom
apetrov/fix-input-autofill

Conversation

@adrianptrv

Copy link
Copy Markdown
Contributor

Closes #17595

Description

Same changes as in: IgniteUI/igniteui-webcomponents#2298

Type of Change (check all that apply):

  • Bug fix

Component(s) / Area(s) Affected:

Input Group

How Has This Been Tested?

  • Manual testing

Checklist:

  • All relevant labels have been applied to this PR

@adrianptrv
adrianptrv requested a review from simeonoff October 22, 2026 16:06
Copilot AI balanced review requested due to automatic review settings October 22, 2026 16:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Autofilled inputs bypass the disabled text color because the shared disabled-state rules do not update the new custom property.

1 open issue
What changed in this PR

This PR addresses #17595 by adjusting Input Group autofill styling so browser backgrounds do not obscure floating labels in border mode.

Changes:

  • Clips autofill backgrounds to text and explicitly sets the autofilled text color.
  • Uses a shared custom property for default, hover, and focused text colors.
File Description
projects/​igniteui-angular/​input-group/​src/​input-group/​themes/​_base.scss Adds autofill styling and centralizes input text-color state handling.

Review effort: Balanced

@simeonoff simeonoff 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.

Nice work on this one, thanks! Moving the text color into a single variable that every state updates keeps the autofilled text matching the theme on hover, focus, readonly and disabled, and background-clip: text fixes the label overlap without touching the rest of the input. It's also good to have this match the web components fix.

@simeonoff
simeonoff enabled auto-merge (squash) October 28, 2026 13:32
@simeonoff
simeonoff merged commit e256830 into master Oct 28, 2026
7 checks passed
@simeonoff
simeonoff deleted the apetrov/fix-input-autofill branch October 28, 2026 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Input Group] Autocomplete background overlaps floating label with border type

3 participants