-
Notifications
You must be signed in to change notification settings - Fork 8
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 bug with hidden vertical input mode. #370
Fix bug with hidden vertical input mode. #370
Conversation
@SterlingButters would you like to review this one? |
@MicahGale I trust you 😎 |
Thanks for the confidence @SterlingButters. Though the expectation is that no code gets merged without someone besides the author reviewing it, no matter what. The general things we look for are covered by the merge checklist and the developer philosophy. So would you be willing to complete this required review? If not, no worries we can have someone else review it. |
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.
Looks good to me
Thanks for the review. |
…al-input-is-behind-comment
Vertical mode detection accidentally wasn't considering comments. This fixed this logic.
Fixes #370.