Skip to content

Conversation

@billyvg
Copy link
Member

@billyvg billyvg commented Apr 28, 2023

Define custom elements so that we the :defined pseudo-class works as it can be used as a "loading" indicator while the web component is being loaded.

Fixes getsentry/sentry-javascript#7988

Define custom elements so that we the `:defined` pseudo-class works as it can be used as a "loading" indicator while the web component is being loaded.

Fixes getsentry/sentry-javascript#7988
@billyvg billyvg force-pushed the feat-replay-define-custom-elements-shadow-om branch from ed0133c to 49eaad5 Compare April 28, 2023 15:05
@billyvg billyvg requested a review from mydea April 28, 2023 15:08
@billyvg billyvg marked this pull request as ready for review April 28, 2023 15:08
@mydea
Copy link
Member

mydea commented May 2, 2023

Hmm, could we add one or two tests for this? 🤔

@billyvg
Copy link
Member Author

billyvg commented May 4, 2023

@mydea added a test

@billyvg billyvg merged commit 7fb1eb8 into sentry-v1 May 5, 2023
@billyvg billyvg deleted the feat-replay-define-custom-elements-shadow-om branch May 5, 2023 17:34
mydea pushed a commit to getsentry/sentry-javascript that referenced this pull request May 8, 2023
- fix: Fix some input masking (esp for radio buttons) (getsentry/rrweb#85)
- fix: Unescaped `:` in CSS rule from Safari (getsentry/rrweb#86)
- feat: Define custom elements (web components) (getsentry/rrweb#87)
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.

:defined CSS pseudo class does not work on shadow DOM elements

3 participants