-
Notifications
You must be signed in to change notification settings - Fork 186
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
Prop Descriptions #52
base: master
Are you sure you want to change the base?
Conversation
829c218
to
e7dc3d8
Compare
@markdalgleish this should be all good now |
Any chance you could add an example that exercises this? Right now, it's hard for me to verify this is working as desired. |
src/Playroom/CodeMirror-JSX.js
Outdated
@@ -0,0 +1,306 @@ | |||
/* eslint-disable new-cap */ |
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.
Is this file borrowed from somewhere else? Seems like a copy-and-paste.
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.
It was the addon-xml 😅 I went back for a second look and was able to use that code once it was registered to codemirror and just handle the tooltip stuff myself.
I switched to rendering the tooltip with lit-html. I can remove that and switch back to document methods if you want, but I think the code is a little clearer this way.
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.
Actually I realized i could just use react and it become much clearer 👍
Added descriptions and defaults to the typescript example. |
05b9cad
to
f5a4186
Compare
@markdalgleish Does your major re-architecture effect any of this? I'm happy to change what ever else you need me to. Really excited to get this and #55 in. EDIT: Doesn't look like our changes conflict much. I mostly just edited |
If these changes are too much a plugin API for suggestions would also probably fit my use case. As it is playroom is only useful for people that are super familiar with the components and how they work together or have the docs up in a separate tab. Even when i try to use playroom myself I feel crippled without having the type definitions helping me along. |
would be nice to merge this feature <3 |
closes #37
I lifted the code mostly from code mirror addon xml