haskal
/
jiti-meet
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

rn: adds support for mobile youtube link

This commit is contained in:
Titus-Andrei Moldovan 2020-06-17 15:01:21 +03:00 committed by Saúl Ibarra Corretgé
parent e87167dd2d
commit 482ba23954
2 changed files with 15 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
react/features/youtube-player/components/AbstractEnterVideoLinkPrompt.js
View File

@ -76,7 +76,7 @@ export default class AbstractEnterVideoLinkPrompt<S: *> extends Component < Prop
* @returns {boolean}
*/
function getYoutubeLink(url) {
const p = /^(?:https?:\/\/)?(?:www\.)?(?:youtu\.be\/|youtube\.com\/(?:embed\/|v\/|watch\?v=|watch\?.+&v=))((\w|-){11})(?:\S+)?$/;// eslint-disable-line max-len
const p = /^(?:https?:\/\/)?(?:www\.)?(?:youtu\.be\/|(?:m\.)?youtube\.com\/(?:embed\/|v\/|watch\?v=|watch\?.+&v=))((\w|-){11})(?:\S+)?$/;// eslint-disable-line max-len
const result = url.match(p);
return result ? result[1] : false;

28
react/features/youtube-player/components/native/YoutubeLargeVideo.js
View File

@ -45,6 +45,13 @@ type Props = {
*/
_isPlaying: string,
/**
* Set to true when the status is set to stop and the view should not react to further changes.
*
* @private
*/
_isStopped: boolean,
/**
* True if in landscape mode.
*
@ -101,13 +108,6 @@ type Props = {
*/
_seek: number,
/**
* Set to true when the status is set to stop and the view should not react to further changes.
*
* @private
*/
_shouldPrepareForStop: boolean,
/**
* Youtube id of the video to be played.
*
@ -138,11 +138,11 @@ const YoutubeLargeVideo = (props: Props) => {
const {
_isOwner,
_isPlaying,
_shouldPrepareForStop,
_isStopped,
_seek
} = props;
if (shouldSetNewStatus(_shouldPrepareForStop, _isOwner, e, _isPlaying, time, _seek)) {
if (shouldSetNewStatus(_isStopped, _isOwner, e, _isPlaying, time, _seek)) {
props._onVideoChangeEvent(props.youtubeId, e, time, props._ownerId);
}
});
@ -201,7 +201,7 @@ const YoutubeLargeVideo = (props: Props) => {
* Return true if the user is the owner and
* the status has changed or the seek time difference from the previous set is larger than 5 seconds.
*
* @param {boolean} shouldPrepareForStop - Once the status was set to stop, all the other statuses should be ignored.
* @param {boolean} isStopped - Once the status was set to stop, all the other statuses should be ignored.
* @param {boolean} isOwner - Whether the local user is sharing the video.
* @param {string} status - The new status.
* @param {boolean} isPlaying - Whether the component is playing at the moment.
@ -210,8 +210,8 @@ const YoutubeLargeVideo = (props: Props) => {
* @private
* @returns {boolean}
*/
function shouldSetNewStatus(shouldPrepareForStop, isOwner, status, isPlaying, newTime, previousTime) {
if (shouldPrepareForStop) {
function shouldSetNewStatus(isStopped, isOwner, status, isPlaying, newTime, previousTime) {
if (isStopped) {
return false;
}
@ -256,12 +256,12 @@ function _mapStateToProps(state) {
_enableControls: ownerId === localParticipant.id,
_isOwner: ownerId === localParticipant.id,
_isPlaying: status === 'playing',
_isStopped: status === 'stop',
_isWideScreen: responsiveUi.aspectRatio === ASPECT_RATIO_WIDE,
_ownerId: ownerId,
_screenHeight: screenHeight,
_screenWidth: screenWidth,
_seek: time,
_shouldPrepareForStop: status === 'stop'
_seek: time
};
}