Change design of toggle button

merge-requests/105/head
Sean King 4 years ago
parent 03e33df86b
commit 796bf9c844

@ -9,8 +9,14 @@ export default class SettingToggle extends React.PureComponent {
prefix: PropTypes.string, prefix: PropTypes.string,
settings: ImmutablePropTypes.map.isRequired, settings: ImmutablePropTypes.map.isRequired,
settingPath: PropTypes.array.isRequired, settingPath: PropTypes.array.isRequired,
label: PropTypes.node.isRequired, label: PropTypes.node,
onChange: PropTypes.func.isRequired, onChange: PropTypes.func.isRequired,
icons: PropTypes.oneOfType([
PropTypes.bool,
PropTypes.object,
]),
condition: PropTypes.string,
ariaLabel: PropTypes.string,
} }
onChange = ({ target }) => { onChange = ({ target }) => {
@ -18,13 +24,13 @@ export default class SettingToggle extends React.PureComponent {
} }
render() { render() {
const { prefix, settings, settingPath, label } = this.props; const { prefix, settings, settingPath, label, icons, condition, ariaLabel } = this.props;
const id = ['setting-toggle', prefix, ...settingPath].filter(Boolean).join('-'); const id = ['setting-toggle', prefix, ...settingPath].filter(Boolean).join('-');
return ( return (
<div className='setting-toggle'> <div className='setting-toggle' aria-label={ariaLabel}>
<Toggle id={id} checked={settings.getIn(settingPath)} onChange={this.onChange} onKeyDown={this.onKeyDown} /> <Toggle id={id} checked={condition ? settings.getIn(settingPath) === condition : settings.getIn(settingPath)} onChange={this.onChange} icons={icons} onKeyDown={this.onKeyDown} />
<label htmlFor={id} className='setting-toggle__label'>{label}</label> {label && (<label htmlFor={id} className='setting-toggle__label'>{label}</label>)}
</div> </div>
); );
} }

@ -13,6 +13,7 @@ import { openModal } from '../../../actions/modal';
import { openSidebar } from '../../../actions/sidebar'; import { openSidebar } from '../../../actions/sidebar';
import Icon from '../../../components/icon'; import Icon from '../../../components/icon';
import { changeSetting, getSettings } from 'soapbox/actions/settings'; import { changeSetting, getSettings } from 'soapbox/actions/settings';
import SettingToggle from '../../notifications/components/setting_toggle';
const messages = defineMessages({ const messages = defineMessages({
post: { id: 'tabs_bar.post', defaultMessage: 'Post' }, post: { id: 'tabs_bar.post', defaultMessage: 'Post' },
@ -31,7 +32,7 @@ class TabsBar extends React.PureComponent {
toggleTheme: PropTypes.func, toggleTheme: PropTypes.func,
logo: PropTypes.string, logo: PropTypes.string,
account: ImmutablePropTypes.map, account: ImmutablePropTypes.map,
themeMode: PropTypes.string, settings: ImmutablePropTypes.map.isRequired,
} }
state = { state = {
@ -83,18 +84,12 @@ class TabsBar extends React.PureComponent {
})); }));
} }
getNewThemeValue() {
if (this.props.themeMode === 'light') return 'dark';
return 'light';
}
handleToggleTheme = () => { handleToggleTheme = () => {
this.props.toggleTheme(this.getNewThemeValue()); this.props.toggleTheme(this.props.settings.get('themeMode') === 'light' ? 'dark' : 'light');
} }
render() { render() {
const { account, onOpenCompose, onOpenSidebar, intl, themeMode } = this.props; const { account, onOpenCompose, onOpenSidebar, intl, settings } = this.props;
const { collapsed } = this.state; const { collapsed } = this.state;
const classes = classNames('tabs-bar', { const classes = classNames('tabs-bar', {
@ -113,9 +108,7 @@ class TabsBar extends React.PureComponent {
</div> </div>
{ account && { account &&
<div className='flex'> <div className='flex'>
<button className='tabs-bar__button-theme-toggle button' onClick={this.handleToggleTheme} aria-label={themeMode === 'light' ? intl.formatMessage(messages.switchToDark) : intl.formatMessage(messages.switchToLight)}> <SettingToggle settings={settings} settingPath={['themeMode']} condition={'light'} onChange={this.handleToggleTheme} icons={{ checked: <Icon id='sun' />, unchecked: <Icon id='moon' /> }} ariaLabel={settings.get('themeMode') === 'light' ? intl.formatMessage(messages.switchToDark) : intl.formatMessage(messages.switchToLight)} />
<Icon id={themeMode === 'light' ? 'sun' : 'moon'} />
</button>
<div className='tabs-bar__profile'> <div className='tabs-bar__profile'>
<Avatar account={account} /> <Avatar account={account} />
<button className='tabs-bar__sidebar-btn' onClick={onOpenSidebar} /> <button className='tabs-bar__sidebar-btn' onClick={onOpenSidebar} />
@ -151,7 +144,7 @@ const mapStateToProps = state => {
return { return {
account: state.getIn(['accounts', me]), account: state.getIn(['accounts', me]),
logo: state.getIn(['soapbox', 'logo']), logo: state.getIn(['soapbox', 'logo']),
themeMode: settings.get('themeMode'), settings: settings,
}; };
}; };

@ -149,6 +149,25 @@
display: flex; display: flex;
} }
.setting-toggle {
margin-top: 5px;
margin-left: 10px;
.react-toggle-track-check,
.react-toggle-track-x {
height: 15px;
color: #fff;
}
.react-toggle--checked {
.react-toggle-track {
background-color: var(--accent-color);
}
&:hover:not(.react-toggle--disabled) .react-toggle-track {
background-color: var(--accent-color--bright);
}
}
}
} }
.tabs-bar__link { .tabs-bar__link {

Loading…
Cancel
Save