PLT-3242 Fixed clicking on input labels on iOS (#3417)

* Prevented channel notification settings radiobuttons from multi-selecting

* Fixed clicking on input labels on iOS
This commit is contained in:
Harrison Healey
2016-06-27 08:21:24 -04:00
committed by Christopher Speller
parent 459eee85a9
commit b9cf09b5a4
2 changed files with 12 additions and 0 deletions

View File

@@ -132,6 +132,7 @@ export default class ChannelNotificationsModal extends React.Component {
<label>
<input
type='radio'
name='desktopNotificationLevel'
checked={notifyActive[0]}
onChange={this.handleUpdateNotifyLevel.bind(this, 'default')}
/>
@@ -149,6 +150,7 @@ export default class ChannelNotificationsModal extends React.Component {
<label>
<input
type='radio'
name='desktopNotificationLevel'
checked={notifyActive[1]}
onChange={this.handleUpdateNotifyLevel.bind(this, 'all')}
/>
@@ -160,6 +162,7 @@ export default class ChannelNotificationsModal extends React.Component {
<label>
<input
type='radio'
name='desktopNotificationLevel'
checked={notifyActive[2]}
onChange={this.handleUpdateNotifyLevel.bind(this, 'mention')}
/>
@@ -171,6 +174,7 @@ export default class ChannelNotificationsModal extends React.Component {
<label>
<input
type='radio'
name='desktopNotificationLevel'
checked={notifyActive[3]}
onChange={this.handleUpdateNotifyLevel.bind(this, 'none')}
/>
@@ -285,6 +289,7 @@ export default class ChannelNotificationsModal extends React.Component {
<label>
<input
type='radio'
name='markUnreadLevel'
checked={this.state.unreadLevel === 'all'}
onChange={this.handleUpdateMarkUnreadLevel.bind(this, 'all')}
/>
@@ -299,6 +304,7 @@ export default class ChannelNotificationsModal extends React.Component {
<label>
<input
type='radio'
name='markUnreadLevel'
checked={this.state.unreadLevel === 'mention'}
onChange={this.handleUpdateMarkUnreadLevel.bind(this, 'mention')}
/>

View File

@@ -1051,3 +1051,9 @@
}
}
}
// on iOS, allow clicks within an input's label to actually propagate through to the input itself
// http://stackoverflow.com/a/34810294/6325807
label span {
pointer-events: none;
}