Skip to content

Conversation

desig9stein
Copy link
Contributor

@desig9stein desig9stein commented Jun 26, 2025

Closes #15775

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

@desig9stein desig9stein removed the ❌ status: awaiting-test PRs awaiting manual verification label Jun 26, 2025
@desig9stein desig9stein marked this pull request as ready for review June 26, 2025 11:51
@desig9stein desig9stein marked this pull request as draft July 2, 2025 05:29
@desig9stein desig9stein marked this pull request as ready for review July 3, 2025 11:18
@simeonoff simeonoff added the 💥 status: in-test PRs currently being tested label Jul 7, 2025
@simeonoff
Copy link
Collaborator

@desig9stein Please add the web components file input in the input group sample for comparison.

@simeonoff
Copy link
Collaborator

simeonoff commented Jul 7, 2025

Looks like there's a bug in the Boostrap theme where an additional border appears:

Screenshot 2025-07-07 at 10 59 09

In Fluent, the Browse Button text jumps upon focus:

Screen.Recording.2025-07-07.at.11.02.10.mov

@desig9stein desig9stein requested a review from simeonoff July 7, 2025 12:58
@simeonoff
Copy link
Collaborator

simeonoff commented Jul 10, 2025

@desig9stein The issue in bootstrap is even more severe with the latest commits.

@simeonoff
Copy link
Collaborator

@desig9stein Now the textarea component jumps on focus. It's a regression compared to the 20.0.x branch.

@simeonoff simeonoff added ✅ status: verified Applies to PRs that have passed manual verification and removed 💥 status: in-test PRs currently being tested labels Jul 23, 2025
@simeonoff simeonoff merged commit 0c1ad42 into 20.0.x Jul 23, 2025
5 checks passed
@simeonoff simeonoff deleted the mpopov/refactoring/input-group/file-input branch July 23, 2025 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

input input-group version: 20.0.x ✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Input Group: Align the design of the input type="file" to the one in Ignite UI for Web Components

2 participants