-
-
Notifications
You must be signed in to change notification settings - Fork 34
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
Link copied feedback and some styling customisation would be sweet! #11
Comments
I appreciate you taking time to provide valuable feedback
Yeah, I also wanted some to add some proper UI feedback but my primary goal was to keep it minimal with zero dependency on any styling library, (Anyways I'm happy to reconsider adding some minimal styling library like https://github.com/cristianbote/goober) but this will still add 1KB(gz) to runtime which is too much for a 3KB(gz) Library
I think making it default export might be better since there are no other component coming in future, I'll think of doing this in next major release |
I think you could easily achieve this by simply
You can do this without any library, it's just a bit jarring right now. |
Sounds good, If I get some time in upcoming days I'll try to incorporate this suggestions |
Hey this is really nice.
A few suggestions
!important
to override the styling for the UI. Let me know if there's a better wayR
, kind of non standard. I would simply change it toimport { ReactWebShare } from 'react-web-share'
or make it a default export.No big deal either way, this is really helpful!
The text was updated successfully, but these errors were encountered: