Skip to content

Conversation

devversion
Copy link
Member

@devversion devversion commented Jun 16, 2021

Here are examples for the non-MDC form field:

Not focused outline
outline_before

Focused outline
Capture

…or in high-contrast mode

Sets up a stronger focus indicator for form fields in high-contrast
mode.
@devversion devversion added Accessibility This issue is related to accessibility (a11y) target: patch This PR is targeted for the next patch release merge: preserve commits When the PR is merged, a rebase and merge should be performed labels Jun 16, 2021
@devversion devversion requested a review from crisbeto June 16, 2021 18:09
@devversion devversion requested a review from mmalerba as a code owner June 16, 2021 18:09
@google-cla google-cla bot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Jun 16, 2021
…nce in high-contrast

In high-contast mode, form fields in outline appearance do not have
any focus indication. This commit adds a focus indicator in
high-contrast mode. Additionally, the focus indication for filled
form fields has been improved. Instead of relying on the underline
ripple that overlaps with the outline, we thicken the outline border
on focus (similar as with the outline appearance).

Fixes angular#22989.
@devversion devversion force-pushed the fix/form-field-high-contrast-focus branch from 770da63 to 7c15335 Compare June 17, 2021 09:03
@devversion devversion added the action: merge The PR is ready for merge by the caretaker label Jun 21, 2021
@andrewseguin andrewseguin merged commit 427bbdd into angular:master Jul 2, 2021
@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 Aug 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Accessibility This issue is related to accessibility (a11y) action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement merge: preserve commits When the PR is merged, a rebase and merge should be performed target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants