[Glitch] `frequentlyUsedLanguages` not updated correctly
Port 8adf67f2db
to glitch-soc
Signed-off-by: Claire <claire.github-309c@sitedethib.com>
pull/2834/head
parent
663cebbe3b
commit
8e3c47aff5
|
@ -1,12 +0,0 @@
|
||||||
import { saveSettings } from './settings';
|
|
||||||
|
|
||||||
export const LANGUAGE_USE = 'LANGUAGE_USE';
|
|
||||||
|
|
||||||
export const useLanguage = language => dispatch => {
|
|
||||||
dispatch({
|
|
||||||
type: LANGUAGE_USE,
|
|
||||||
language,
|
|
||||||
});
|
|
||||||
|
|
||||||
dispatch(saveSettings());
|
|
||||||
};
|
|
|
@ -240,7 +240,6 @@ class LanguageDropdown extends PureComponent {
|
||||||
frequentlyUsedLanguages: PropTypes.arrayOf(PropTypes.string),
|
frequentlyUsedLanguages: PropTypes.arrayOf(PropTypes.string),
|
||||||
intl: PropTypes.object.isRequired,
|
intl: PropTypes.object.isRequired,
|
||||||
onChange: PropTypes.func,
|
onChange: PropTypes.func,
|
||||||
onClose: PropTypes.func,
|
|
||||||
};
|
};
|
||||||
|
|
||||||
state = {
|
state = {
|
||||||
|
@ -257,14 +256,11 @@ class LanguageDropdown extends PureComponent {
|
||||||
};
|
};
|
||||||
|
|
||||||
handleClose = () => {
|
handleClose = () => {
|
||||||
const { value, onClose } = this.props;
|
|
||||||
|
|
||||||
if (this.state.open && this.activeElement) {
|
if (this.state.open && this.activeElement) {
|
||||||
this.activeElement.focus({ preventScroll: true });
|
this.activeElement.focus({ preventScroll: true });
|
||||||
}
|
}
|
||||||
|
|
||||||
this.setState({ open: false });
|
this.setState({ open: false });
|
||||||
onClose(value);
|
|
||||||
};
|
};
|
||||||
|
|
||||||
handleChange = value => {
|
handleChange = value => {
|
||||||
|
|
|
@ -4,7 +4,6 @@ import { connect } from 'react-redux';
|
||||||
|
|
||||||
|
|
||||||
import { changeComposeLanguage } from 'flavours/glitch/actions/compose';
|
import { changeComposeLanguage } from 'flavours/glitch/actions/compose';
|
||||||
import { useLanguage } from 'flavours/glitch/actions/languages';
|
|
||||||
|
|
||||||
import LanguageDropdown from '../components/language_dropdown';
|
import LanguageDropdown from '../components/language_dropdown';
|
||||||
|
|
||||||
|
@ -28,11 +27,6 @@ const mapDispatchToProps = dispatch => ({
|
||||||
dispatch(changeComposeLanguage(value));
|
dispatch(changeComposeLanguage(value));
|
||||||
},
|
},
|
||||||
|
|
||||||
onClose (value) {
|
|
||||||
// eslint-disable-next-line react-hooks/rules-of-hooks -- this is not a react hook
|
|
||||||
dispatch(useLanguage(value));
|
|
||||||
},
|
|
||||||
|
|
||||||
});
|
});
|
||||||
|
|
||||||
export default connect(mapStateToProps, mapDispatchToProps)(LanguageDropdown);
|
export default connect(mapStateToProps, mapDispatchToProps)(LanguageDropdown);
|
||||||
|
|
|
@ -1,8 +1,8 @@
|
||||||
import { Map as ImmutableMap, fromJS } from 'immutable';
|
import { Map as ImmutableMap, fromJS } from 'immutable';
|
||||||
|
|
||||||
import { COLUMN_ADD, COLUMN_REMOVE, COLUMN_MOVE, COLUMN_PARAMS_CHANGE } from '../actions/columns';
|
import { COLUMN_ADD, COLUMN_REMOVE, COLUMN_MOVE, COLUMN_PARAMS_CHANGE } from '../actions/columns';
|
||||||
|
import { COMPOSE_LANGUAGE_CHANGE } from '../actions/compose';
|
||||||
import { EMOJI_USE } from '../actions/emojis';
|
import { EMOJI_USE } from '../actions/emojis';
|
||||||
import { LANGUAGE_USE } from '../actions/languages';
|
|
||||||
import { LIST_DELETE_SUCCESS, LIST_FETCH_FAIL } from '../actions/lists';
|
import { LIST_DELETE_SUCCESS, LIST_FETCH_FAIL } from '../actions/lists';
|
||||||
import { NOTIFICATIONS_FILTER_SET } from '../actions/notifications';
|
import { NOTIFICATIONS_FILTER_SET } from '../actions/notifications';
|
||||||
import { SETTING_CHANGE, SETTING_SAVE } from '../actions/settings';
|
import { SETTING_CHANGE, SETTING_SAVE } from '../actions/settings';
|
||||||
|
@ -182,7 +182,7 @@ export default function settings(state = initialState, action) {
|
||||||
return changeColumnParams(state, action.uuid, action.path, action.value);
|
return changeColumnParams(state, action.uuid, action.path, action.value);
|
||||||
case EMOJI_USE:
|
case EMOJI_USE:
|
||||||
return updateFrequentEmojis(state, action.emoji);
|
return updateFrequentEmojis(state, action.emoji);
|
||||||
case LANGUAGE_USE:
|
case COMPOSE_LANGUAGE_CHANGE:
|
||||||
return updateFrequentLanguages(state, action.language);
|
return updateFrequentLanguages(state, action.language);
|
||||||
case SETTING_SAVE:
|
case SETTING_SAVE:
|
||||||
return state.set('saved', true);
|
return state.set('saved', true);
|
||||||
|
|
Loading…
Reference in New Issue