Skip to content

[InputFile] [a11y] Fix missing focus indicator in FluentInputFile (#3693)#3722

Merged
vnbaaij merged 1 commit intomicrosoft:devfrom
MarvinKlein1508:fix-3693
Apr 23, 2025
Merged

[InputFile] [a11y] Fix missing focus indicator in FluentInputFile (#3693)#3722
vnbaaij merged 1 commit intomicrosoft:devfrom
MarvinKlein1508:fix-3693

Conversation

@MarvinKlein1508
Copy link
Copy Markdown
Collaborator

Pull Request

📖 Description

This PR fixes the missing focus indicator in FluentInputFile.

Before:
before

After:
after

🎫 Issues

Fixes #3693

👩‍💻 Reviewer Notes

This issue is being caused by the input field having opacity set to 0. This results in the UI not showing any focus borders. This PR fixes this by giving the parent container the corresponding outline.

📑 Test Plan

✅ Checklist

General

  • I have added tests for my changes.
  • I have tested my changes.
  • I have updated the project documentation to reflect my changes.
  • I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

  • I have added a new component
  • I have added Unit Tests for my new component
  • I have modified an existing component
  • I have validated the Unit Tests for an existing component

@MarvinKlein1508 MarvinKlein1508 changed the title [InputFile] [a11y] Fix missing focus indicator in FluentInputFile [InputFile] [a11y] Fix missing focus indicator in FluentInputFile (#3693) Apr 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Visible Focus Indicator Not Appearing on "Drag files here/Browse" Element

3 participants