-
-
Notifications
You must be signed in to change notification settings - Fork 221
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
Add @types/Glightbox #405
Comments
Hey there @tomasvn Currently, adding things like types or other typescript adjacent things isn't a high priority. That said, if you are able to get the ball rolling, I'd be more than happy to help out/check your work. |
@gingerchew Hi, I am not that well versed in TS, but I can try it. |
If you want, we can start at creating types/interface for the config object (I imagine that's what people would want typing for the most) and then work from there. |
@gingerchew this is my draft for DefinitelyTyped package or the types should be here in the main repo? |
That's a good question, I'm leaning towards just adding an |
The typescript in the link you gave looks good to me, I wonder if there's a way to get the types from Plyr so that we don't need to constantly update our own in parallel. (This may be wishful thinking though) |
Install plyr types as peerDep? So plyr options could be imported? |
I couldn't find a I'm also leaning away from peerDeps, I see them causing more trouble than benefit (maybe biased by personal experience.) |
I did find them, but it seems that is has different options https://github.com/sampotts/plyr/blob/master/src/js/plyr.d.ts #411 - Draft PR |
You can actually pass the plyr config to glightbox,the ones documented here are just the ones you're most likely to use. |
Hi, I am rather new to Angular and TypeScript. I found this library and it is exactly what I need. However, for using it I would need the types. Can you tell me if this is provided soon or give me some information on how to include the types definition into a Angular project? |
@rokozeit There is an opened PR that is still in progress |
Please describe the feature you want to be implemented.
Add types for the package
The text was updated successfully, but these errors were encountered: