Skip to content

Improve shadow DOM focus rects#29750

Merged
spmonahan merged 8 commits intomicrosoft:shadow-domfrom
spmonahan:merge-styles/better-shadow-focus-rects
Nov 16, 2023
Merged

Improve shadow DOM focus rects#29750
spmonahan merged 8 commits intomicrosoft:shadow-domfrom
spmonahan:merge-styles/better-shadow-focus-rects

Conversation

@spmonahan
Copy link
Contributor

@spmonahan spmonahan commented Nov 4, 2023

Moves the FocusRectsProvider to be outside of the shadow DOM, removing a <div> inside the shadow root. Now the FocusRects class name is applied to the shadow root host element and the focus visibility styles are applied by using the :host() selector on the shadow root.

NOTE: this does not address all focus visibility as some components define these styles outside of the changes here. Those components will be updated in a follow up PR.

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.
@spmonahan spmonahan requested review from a team, dzearing and phkuo as code owners November 4, 2023 00:50
@codesandbox-ci
Copy link

codesandbox-ci bot commented Nov 4, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 946ea04:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@spmonahan spmonahan requested a review from a team as a code owner November 15, 2023 21:20
@spmonahan spmonahan requested a review from a team as a code owner November 15, 2023 21:46
@spmonahan spmonahan merged commit d6be7fe into microsoft:shadow-dom Nov 16, 2023
spmonahan added a commit that referenced this pull request Nov 16, 2023
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Dec 5, 2023
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Dec 15, 2023
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit that referenced this pull request Jan 4, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Jan 4, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Jan 5, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Feb 16, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Feb 16, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Feb 22, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Feb 23, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Feb 26, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Feb 26, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Feb 28, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Mar 1, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit that referenced this pull request Mar 8, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Mar 8, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit that referenced this pull request Mar 23, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit that referenced this pull request Mar 26, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit that referenced this pull request Mar 28, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit to spmonahan/fluentui that referenced this pull request Mar 29, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit that referenced this pull request Apr 4, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit that referenced this pull request Apr 15, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
spmonahan added a commit that referenced this pull request Apr 18, 2024
* fix: improve focus rects support in shadow DOM

Moves the FocusRectsProvider to be outside of the shadow DOM, removing
a <div> inside the shadow root. Now the FocusRects class name is applied
to the shadow root host element and the focus visibility styles are applied
by using the `:host()` selector on the shadow root.

* update snapshots

* link: supports focus rects in shadow dom

* checkbox: supports focus rects in shadow dom

* update styles to support focus rects in shadow dom

* update snapshots

* update snapshots

* Updating snapshots.

---------

Co-authored-by: KHMakoto <Humberto.Morimoto@microsoft.com>
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.

2 participants