Skip to content

Conversation

piti118
Copy link

@piti118 piti118 commented Sep 12, 2017

I think this gives a better ux since schema can be guessed from example and it tends to be way longer than the body itself. Having it displayed by default is a bit distracting.

#245

I think this gives a better ux since schema can be guessed from example and it tends to be way longer than the body itself. Having it displayed by default is a bit distracting.
@coveralls
Copy link

coveralls commented Sep 12, 2017

Coverage Status

Coverage remained the same at 58.211% when pulling 8e48db0 on piti118:patch-1 into c52a482 on danielgtaylor:olio-theme.

@w0k3
Copy link

w0k3 commented Sep 14, 2018

@danielgtaylor will this ever get pulled? The patch looks fine.

@bluenex
Copy link

bluenex commented Jan 23, 2019

I have tested this and found that I can't expand the schema as the parent height is already fixed while schema height is still 0.

I still can't figure how to workaround this though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants