JoinAudio Modal UI Update #3534

Merged
merged 11 commits into from Jan 10, 2017

Projects

None yet

3 participants

@KDSBrowne
Contributor

JoinAudio with mic -> updated the menu styling.
JoinAudio listenOnly -> no longer has a menu and just joins listen only.

@@ -4,6 +4,7 @@ import { clearModal } from '/imports/ui/components/app/service';
import classNames from 'classnames';
import ReactDOM from 'react-dom';
import styles from '../styles.scss';
+import { joinListenOnly } from '/imports/api/phone';
@oswaldoacauan
oswaldoacauan Dec 23, 2016 Member

Should me moved to the container and passed down as a prop.

</div>
<div className={styles.containerRightHalfContent}>
+ <span className={styles.heading}>Speaker source</span>
+ <DeviceSelector
@oswaldoacauan
oswaldoacauan Dec 23, 2016 Member

Wrong indentation.

@@ -9,6 +9,7 @@ import styles from './styles.scss';
import JoinAudio from './join-audio/component';
import ListenOnly from './listen-only/component';
import AudioSettings from './audio-settings/component';
+import { joinListenOnly } from '/imports/api/phone';
@oswaldoacauan
oswaldoacauan Dec 23, 2016 edited Member

Only Container components should import and perform API actions.

@antobinary antobinary merged commit 0b739ad into bigbluebutton:master Jan 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment