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

[Feature Request] Add TypeScript typings for argon-design-system-react package #5

Closed
rjkz808 opened this issue Feb 13, 2020 · 11 comments
Labels
enhancement New feature or request

Comments

@rjkz808
Copy link

rjkz808 commented Feb 13, 2020

What is your enhancement?

This UI kit is awesome. But it could be even better if there's some possibility to integrate with TypeScript. A lot of React projects use TS and also a lot of react component libraries have their own types declaration files (for example, https://github.com/react-bootstrap/react-bootstrap and https://github.com/mui-org/material-ui).

@einazare einazare added the enhancement New feature or request label Feb 14, 2020
@einazare
Copy link
Contributor

Hello there, @rjkz808 ,

I've just made a new branch called feature/typescript-support: https://github.com/creativetimofficial/argon-design-system-react/tree/feature/typescript-support
If you want, you can create a PR there, but you will have to keep the actual code base, i.e., the project should work with Babel as well as TypeScript.
Also, the commits should follow the Angular Contribution Guide: https://github.com/angular/angular/blob/master/CONTRIBUTING.md#-commit-message-guidelines

Best,
Manu

@rjkz808
Copy link
Author

rjkz808 commented Feb 14, 2020

I understand it. Regular declaration files would only affect projects that use TypeScript, I'll not rewrite the whole code base to TS 😄

@rjkz808
Copy link
Author

rjkz808 commented Feb 14, 2020

Oh, I think I missed up the library source code. I thought you have your own React components, but now I understand that it's more like an UI theme for Bootstrap. So, it can be used with any React libraries that are integrated with bootstrap (like react-bootstrap or reactstrap), right?

@rjkz808
Copy link
Author

rjkz808 commented Feb 14, 2020

And this repo is an example of using this theme, isn't it?

@einazare
Copy link
Contributor

Hello again, @rjkz808 ,

I do not follow your comments, especially the last one, which theme are you referring to and which repo?

Best,
Manu

@rjkz808
Copy link
Author

rjkz808 commented Feb 14, 2020

As I understand this repo is a showcase for your Bootstrap CSS theme (which I hope is placed here https://github.com/creativetimofficial/argon-design-system-react/blob/master/src/assets/css/argon-design-system-react.css). I thought that Argon Design System React heading means that you have your own react components library with this styling.

@rjkz808
Copy link
Author

rjkz808 commented Feb 14, 2020

As I understand this repo is a showcase for your Bootstrap CSS theme (which I hope is placed here https://github.com/creativetimofficial/argon-design-system-react/blob/master/src/assets/css/argon-design-system-react.css). I thought that Argon Design System React heading means that you have your own react components library with this styling.

Or maybe it's not a showcase project but a template one

@einazare
Copy link
Contributor

Hello again,

This is a Bootstrap Template: https://demos.creative-tim.com/argon-design-system-react/?ref=adsr-github-readme#/

Best,
Manu

@rjkz808
Copy link
Author

rjkz808 commented Feb 14, 2020

Yup, and this repo is its source code, right?

@rjkz808
Copy link
Author

rjkz808 commented Feb 14, 2020

I mean you don't have your own react components library, but you have some template/showcase repos of your CSS library and one of them is for React. Is it correct?

@einazare
Copy link
Contributor

Yes, indeed.
Please note, that our CSS library, is actually a theme over Bootstrap (or other libraries, such as Tailwindcss, Material-UI, Vuetify, etc.).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants