-
Notifications
You must be signed in to change notification settings - Fork 119
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 log position of compile/upload tabs on first show #1494
Conversation
994e8fa
to
ed69338
Compare
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.
🐛
Expand the Deployment pane and note that top controls (tabs, sidebar buttons) get partially hidden.
ed69338
to
ed03569
Compare
@nkrkv fixed. Check it out, please! |
ed03569
to
874247a
Compare
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.
LGTM
@@ -0,0 +1,98 @@ | |||
/* | |||
Edited fork of `autoscroll-react` from | |||
https://github.com/xodio/autoscroll-react.git#cbb3422461a28b40ca082d6a5f33f667508c3288 |
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.
Nope. The original should be the original original. And a note on what’s been changed:
- onScrollFromBottom
- scrollDown
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 changes essentially fix the bug.
Please, adjust the reference and merge.
874247a
to
c1c3fe7
Compare
No description provided.