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
feat(FEC-13728): [Audio Player Improvements] Add support for download… #18
Conversation
… plugin in audio player
<title>Audio Player Plugin</title> | ||
<script src="http://localhost:8086/kaltura-ovp-player.js"></script> | ||
<script src="http://localhost:8081/playkit-downloads.js"></script> | ||
<!-- <script src="https://unpkg.com/@playkit-js/kaltura-player-js@canary/dist/kaltura-ovp-player.js"></script>--> |
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.
remove comments if not used
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.
fixed
} | ||
} | ||
|
||
:global(.playkit-player.playkit-size-md), |
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.
does this override it for the current player only or for all players on the page ?
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.
fixed
<Tooltip label={moreIconTxt!}> | ||
<A11yWrapper onClick={onClick}> | ||
<button ref={moreButtonRef} className={`${ui.style.upperBarIcon} ${styles.morePluginsIcon}`} tabIndex={0} aria-label={moreIconTxt}> | ||
<Icon id={`${'pluginName-123'}-upper-bar-manager`} path={ICON_PATH} viewBox={'0 0 32 32'} /> |
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.
use a more descriptive id name
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.
fixed
### Description of the Changes Add 'MiniAudioPreset handling #### Resolves FEC-13728 #### Related PR kaltura/playkit-js-audio-player#18 kaltura/playkit-js-downloads#48 --------- Co-authored-by: JonathanTGold <jonathan.gold@86@gmail.com>
#48) …yer to interaction) ### Description of the Changes Expose the download overlay API (to enable Audio Player to interaction) ### Resolves FEC-13728 #### Related PR kaltura/playkit-js-audio-player#18 kaltura/playkit-js-ui#877 --------- Co-authored-by: JonathanTGold <jonathan.gold@86@gmail.com>
### Description of the Changes Add 'MiniAudioPreset handling #### Resolves FEC-13728 #### Related PR kaltura/playkit-js-audio-player#18 kaltura/playkit-js-downloads#48 --------- Co-authored-by: JonathanTGold <jonathan.gold@86@gmail.com>
Description of the Changes
Add support for plugins integration
Resolves FEC-13728
Related PR
kaltura/playkit-js-ui#877
#19
kaltura/playkit-js-share#42
kaltura/playkit-js-downloads#48