-
Notifications
You must be signed in to change notification settings - Fork 1
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
doc(update-guide):Share user guide update #12
base: master
Are you sure you want to change the base?
Conversation
README.md
Outdated
* Whether to add also start postion to the shared media | ||
|
||
#### useNative | ||
* If the share icon is clicked users will have the ability to choose how to share the desired content using plauer custom UI or the native share api |
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.
you mean, this is what happens if useNative is true or...?
* If the share icon is clicked users will have the ability to choose how to share the desired content using plauer custom UI or the native share api | ||
|
||
#### shareOptions | ||
`shareOptions` are optional, the plugin can work only with shareUrl or embedUrl stand alone |
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.
doesn't shareUrl override shareOptions url ?
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.
do not think so
README.md
Outdated
|
||
#### Example | ||
|
||
**[Share Plugin Example](https://codepen.io/giladna/pen/KKoOrLV)** |
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.
I don't know if I would use a codepen link. isn't it possible to host a demo in this project itself ?
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.
we can try
Co-authored-by: Sivan Agranov <88330203+SivanA-Kaltura@users.noreply.github.com>
Co-authored-by: Sivan Agranov <88330203+SivanA-Kaltura@users.noreply.github.com>
Co-authored-by: Sivan Agranov <88330203+SivanA-Kaltura@users.noreply.github.com>
Co-authored-by: Sivan Agranov <88330203+SivanA-Kaltura@users.noreply.github.com>
Co-authored-by: Sivan Agranov <88330203+SivanA-Kaltura@users.noreply.github.com>
Co-authored-by: Sivan Agranov <88330203+SivanA-Kaltura@users.noreply.github.com>
Co-authored-by: Sivan Agranov <88330203+SivanA-Kaltura@users.noreply.github.com>
No description provided.