Skip to content

Conversation

ManicardiFrancesco
Copy link
Contributor

Fixes that form-field-custom-control-example.ts wasn't correctly handling focused events.

This can easily be checked on the official docs page or the stackblitz, here's two sample screenshots:

image
In the above screenshot, the user has focused the input, but the form field outline is not becoming blue.

Another failure case is when you leave the input: in this case, it will stay forever outlined in blue:
image

@ManicardiFrancesco ManicardiFrancesco requested a review from a team as a code owner October 30, 2024 17:43
@ManicardiFrancesco ManicardiFrancesco requested review from crisbeto and andrewseguin and removed request for a team October 30, 2024 17:43
@crisbeto crisbeto added docs This issue is related to documentation action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Oct 31, 2024
@crisbeto crisbeto removed the request for review from andrewseguin October 31, 2024 10:28
@crisbeto crisbeto merged commit 9d4fa8f into angular:main Oct 31, 2024
11 of 13 checks passed
crisbeto pushed a commit that referenced this pull request Oct 31, 2024
crisbeto pushed a commit that referenced this pull request Oct 31, 2024
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Dec 1, 2024
@ManicardiFrancesco ManicardiFrancesco deleted the patch-1 branch December 2, 2024 10:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker docs This issue is related to documentation target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants