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

Link copied feedback and some styling customisation would be sweet! #11

Open
rohan-deshpande opened this issue Jan 20, 2021 · 3 comments

Comments

@rohan-deshpande
Copy link

rohan-deshpande commented Jan 20, 2021

Hey this is really nice.

A few suggestions

  • When using the link copy button, it would be really nice to get a little bit of feedback rather than the dialog just disappearing suddenly
  • It would be nice to get a styling API, right now I've had to use !important to override the styling for the UI. Let me know if there's a better way
  • Just a nitpick with the component name - it's a bit odd to have the React in the name abbreviated to R, kind of non standard. I would simply change it to import { ReactWebShare } from 'react-web-share' or make it a default export.

No big deal either way, this is really helpful!

@harshzalavadiya
Copy link
Member

harshzalavadiya commented Jan 20, 2021

@rohan-deshpande

I appreciate you taking time to provide valuable feedback

When using the link copy button, it would be really nice to get a little bit of feedback rather than the dialog just disappearing suddenly
It would be nice to get a styling API, right now I've had to use !important to override the styling for the UI. Let me know if there's a better way

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

Just a nitpick with the component name - it's a bit odd to have the React in the name abbreviated to R, kind of non standard. I would simply change it to import { ReactWebShare } from 'react-web-share' or make it a default export.

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

@rohan-deshpande
Copy link
Author

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

I think you could easily achieve this by simply

  • Showing the user that the link has been copied via text in the modal
  • Fading out the modal after a small period of time

You can do this without any library, it's just a bit jarring right now.

@harshzalavadiya
Copy link
Member

@rohan-deshpande

Sounds good, If I get some time in upcoming days I'll try to incorporate this suggestions

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

No branches or pull requests

2 participants