Merge pull request #4515 from OZhurbenko/fix-warnings-assign-presenter

[HTML5] - Fix for the React console warnings
This commit is contained in:
Anton Georgiev 2017-10-13 14:06:31 -03:00 committed by GitHub
commit f1de51595d
4 changed files with 13 additions and 7 deletions

View File

@ -28,7 +28,6 @@ const propTypes = {
actionsbar: PropTypes.node,
media: PropTypes.node,
location: PropTypes.object.isRequired,
children: PropTypes.node.isRequired,
};
const defaultProps = {
@ -69,9 +68,7 @@ const AppContainer = (props) => {
actionsbar={actionsbar}
media={media}
{...otherProps}
>
{props.children}
</App>
/>
);
};

View File

@ -120,7 +120,7 @@ class ChatDropdown extends Component {
>
<DropdownTrigger
tabIndex={0}
ariaLabel={intl.formatMessage(intlMessages.options)}
aria-label={intl.formatMessage(intlMessages.options)}
className={styles.btn}
>
<Icon iconName="more" />

View File

@ -41,7 +41,17 @@ export default class DropdownTrigger extends Component {
}
render() {
const { children, className, ...restProps } = this.props;
const remainingProps = { ...this.props };
delete remainingProps.dropdownToggle;
delete remainingProps.dropdownShow;
delete remainingProps.dropdownHide;
const {
children,
className,
...restProps
} = remainingProps;
const TriggerComponent = React.Children.only(children);
const TriggerComponentBounded = React.cloneElement(TriggerComponent, {

View File

@ -191,7 +191,6 @@ class UserParticipants extends Component {
className={styles.scrollableList}
role="tabpanel"
tabIndex={0}
refScrollContainer
ref={(ref) => { this.refScrollContainer = ref; }}
>
<CSSTransitionGroup