-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: sticky chapter on mouseout #1005
Conversation
@luwes is attempting to deploy a commit to the Mux Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1005 +/- ##
==========================================
- Coverage 78.55% 76.94% -1.62%
==========================================
Files 59 50 -9
Lines 11080 12093 +1013
Branches 0 699 +699
==========================================
+ Hits 8704 9305 +601
- Misses 2376 2762 +386
- Partials 0 26 +26 ☔ View full report in Codecov by Sentry. |
@@ -388,6 +387,10 @@ class MediaChromeRange extends globalThis.HTMLElement { | |||
this.#cssRules.pointer = getOrInsertCSSRule(this.shadowRoot, '#pointer'); | |||
this.#cssRules.progress = getOrInsertCSSRule(this.shadowRoot, '#progress'); | |||
this.#cssRules.thumb = getOrInsertCSSRule(this.shadowRoot, '#thumb'); | |||
this.#cssRules.activeSegment = getOrInsertCSSRule( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SMRT.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense as a change per earlier out of band convo.
fix #999
I can't reproduce it anymore so hopefully fixed.
the bug happened when
updateSegments
was called multiple times, easier to reproduce with preload none.insertCSSRule
created a rule every timeupdateSegments
was called so it possibly happened while the user hovered over a chapter.