docs(focus indicators): Add documentation for focus indicators. #18724
Conversation
googlebot
added
the
cla: yes
guides/theming.md
Outdated
| // Including this mixin will theme the focus indicators. | ||
| // This mixin can be included multiple times throughout your app | ||
| // to change the focus indicator's color. | ||
| @include mat-strong-focus-indicators-theme($theme-or-color); |
There was a problem hiding this comment.
Maybe we should use an actual color here like the other examples in this doc?
There was a problem hiding this comment.
Good idea, added an actual color.
guides/theming.md
Outdated
| @@ -251,3 +251,27 @@ function changeTheme(themeName) { | |||
| ### Theming your own components | |||
| For more details about theming your own components, | |||
| see [theming-your-components.md](./theming-your-components.md). | |||
|
|
|||
| ### Showing strong focus indicators | |||
| Angular Material has built-in support for showing **strong focus indicators** on all components. Focus indicators are visual markers that show users which element is focused. Angular Material's strong focus indicators render as highly visible outlines around elements. | |||
There was a problem hiding this comment.
The sentence starting with "Focus indicators..." sounds like it's referring to focus indicators in general, not the strong ones. Maybe we should be more explicit?
There was a problem hiding this comment.
This is purposeful because by "focus indicators" I meant either Material's current focus styles or these new strong ones. Either way, they are visual markers that show users which element is focused. I was also back-and-forth on just removing this sentence if it is just going to cause more confusion.
There was a problem hiding this comment.
I ended up just removing this sentence.
| @@ -251,3 +251,27 @@ function changeTheme(themeName) { | |||
| ### Theming your own components | |||
| For more details about theming your own components, | |||
| see [theming-your-components.md](./theming-your-components.md). | |||
|
|
|||
There was a problem hiding this comment.
Should we mention the mat-focus-indicator class somewhere?
There was a problem hiding this comment.
Added a sentence & example towards the end, let me know if you had something else in mind.
|
Closing this since docs were added in #22374 (I hadn't realized at the time there was an existing PR) |
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
No description provided.