Skip to content

Conversation

SisIvanova
Copy link
Collaborator

@SisIvanova SisIvanova commented Aug 18, 2025

Closes #16026

Additional information (check all that apply):

  • Bug fix
  • New functionality
  • Documentation
  • Demos
  • CI/CD

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code (test guidelines)
  • This PR includes API docs for newly added methods/properties (api docs guidelines)
  • This PR includes feature/README.MD updates for the feature docs
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes ng update migrations for the breaking changes (migrations guidelines)
  • This PR includes behavioral changes and the feature specification has been updated with them

@SisIvanova SisIvanova added the ❌ status: awaiting-test PRs awaiting manual verification label Sep 3, 2025
@SisIvanova SisIvanova marked this pull request as ready for review September 3, 2025 07:47
@@ -8,6 +8,7 @@
<input #input class="input" type="text" igxInput [igxSelectItemNavigation]="this"
[disabled]="disabled"
readonly="true"
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do we need that readonly attr before the [igxReadOnlyInput]="false"

@SisIvanova SisIvanova requested a review from simeonoff September 4, 2025 07:49
@desig9stein
Copy link
Contributor

In the Fluent theme, when the input is type search and read-only, the suffix/prefix icons use an incorrect color. They should look like the read-only igc-input.

Screen.Recording.2025-09-10.at.11.13.04.mov

@desig9stein desig9stein added ✅ status: verified Applies to PRs that have passed manual verification and removed ❌ status: awaiting-test PRs awaiting manual verification labels Sep 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
input-group ✨ themes version: 20.1.x ✅ status: verified Applies to PRs that have passed manual verification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement read-only states for Input Group Components
3 participants