Skip to content
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

Removing top/bottom paddings in CodeMirror-dialog #2697

Merged
merged 3 commits into from
Oct 8, 2020

Conversation

antoine29
Copy link
Contributor

I think the most straightforward way to avoid the CodeMirror searchInputDIalog covering a search result is setting top and bottom paddings to 0.

fix (copy)

Then this PR closes #2646

@netlify
Copy link

netlify bot commented Oct 7, 2020

Deploy preview for insomnia-storybook ready!

Built with commit d9b98e6

https://deploy-preview-2697--insomnia-storybook.netlify.app

@develohpanda develohpanda requested a review from sonicyeti October 7, 2020 02:15
@sonicyeti sonicyeti added C-improvement Category: Improvement / Enhancement B-bug Bug: general classification labels Oct 8, 2020
Copy link
Contributor

@sonicyeti sonicyeti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great catch and call! 👌

@sonicyeti sonicyeti merged commit 0092723 into Kong:develop Oct 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B-bug Bug: general classification C-improvement Category: Improvement / Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Microbug – autoscroll in response preview scrolls badly upwards
2 participants