-
Notifications
You must be signed in to change notification settings - Fork 14
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(FEC-11151): quality dropdown is cropped when user clicks on ‘Quality’ settings on the player after scrolling down #592
Conversation
…ity’ settings on the player after scrolling down
src/components/shell/shell.js
Outdated
eventManager.listen( | ||
document, | ||
'scroll', | ||
debounce(() => { |
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.
remove the anonymous function and use arrow function for the callback _onDocumentScroll
src/components/shell/shell.js
Outdated
@@ -233,7 +240,7 @@ class Shell extends Component { | |||
FakeEvent.Type.RESIZE, | |||
debounce(() => { |
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.
also here
src/components/shell/shell.js
Outdated
* @returns {void} | ||
* @memberof Shell | ||
*/ | ||
_onDocumentScroll(): void { |
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.
If we don't have any more logic in it, no need this function, define just _updatePlayerClientRect
as a callback for the document scroll event.
@yairans don't we need to change in |
No. Gui updated from player |
Description of the Changes
update the player client rect on document.scroll event
CheckLists