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
Fixed issue #3343: Scroll bottom button misbehaviour #3422
Conversation
@@ -10,7 +10,7 @@ const ScrollTopFab = styled(Fab)` | |||
top: ${props => props.height - 95 + 'px'}; | |||
right: ${props => (props.width < 1100 ? '70px' : '100px')}; | |||
margin: 15px; | |||
z-index: 9000; | |||
z-index: 2; |
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.
This change in Z-Index is not required I guess.
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.
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.
The z-index puts the chat modal over the button to resolve this issue.
scrollValues.scrollHeight - Math.ceil(scrollValues.scrollTop) === | ||
scrollValues.clientHeight | ||
!(showScrollBottom && showScrollTop) && | ||
scrollValues.clientHeight + scrollValues.scrollTop < |
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.
add Math.ceil(scrollValues.scrollTop) .
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.
@maze-runnar Done!
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.
@elit-altum resolve conflicts
|
…lay conditions Fixed issue #3343 scroll buttons misbehaving Fixed Issue #3343 scroll buttons misbehaving refactored susi-chatbot.js to ES6 and eliminate the use of jQuery (#3407) Co-authored-by: Akshat Garg <akshatnitd@gmail.com> Made bot wizard stepper linear (#3357) handled check() Co-authored-by: Akshat Garg <akshatnitd@gmail.com> Used class field declarations for javascript (#3419) Co-authored-by: Akshat Garg <akshatnitd@gmail.com> removed blank message (#3416) fixed error in chat with susi (#3440) Used class field declarations for javascript (#3443) Fixed unnecessary scrollbar in settings (#3437) Fixed double scrollbar in settings menu Co-authored-by: Akshat Garg <akshatnitd@gmail.com> Fixed overflow in x-direction (#3424) * overflow in x direction fixed * overflow in x direction fixed Co-authored-by: Akshat Garg <akshatnitd@gmail.com> Sorted the skill feedbacks wrt timestamps (#3433) Co-authored-by: Akshat Garg <akshatnitd@gmail.com> Corrected typo in Projection_Information.md (#3421) Co-authored-by: Akshat Garg <akshatnitd@gmail.com> Checked prettier fails Fixed issue #3343 Scroll button misbehavior
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.
Conflicts are not resolved yet.
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.
Please resolve conflicts.
Conflicts resolved |
Fixes #3343
Changes:
Demo Link : https://pr-3422-fossasia-susi-web-chat.surge.sh/