Merge pull request #1784 from mattermost/plt-1591

PLT-1591 Fix appearance settings scroll bug

1 files changed, 6 insertions, 3 deletions

diff --git a/web/react/components/user_settings/user_settings_appearance.jsx b/web/react/components/user_settings/user_settings_appearance.jsx
index ad41ab771..7bfc9fdbd 100644
--- a/web/react/components/user_settings/user_settings_appearance.jsx
+++ b/web/react/components/user_settings/user_settings_appearance.jsx
@@ -74,6 +74,10 @@ export default class UserSettingsAppearance extends React.Component {
 
         this.props.setEnforceFocus(true);
     }
+    scrollToTop() {
+        $('.ps-container.modal-body').scrollTop(0);
+        $('.ps-container.modal-body').perfectScrollbar('update');
+    }
     submitTheme(e) {
         e.preventDefault();
         var user = UserStore.getCurrentUser();
@@ -88,9 +92,7 @@ export default class UserSettingsAppearance extends React.Component {
 
                 this.props.setRequireConfirm(false);
                 this.originalTheme = Object.assign({}, this.state.theme);
-
-                $('.ps-container.modal-body').scrollTop(0);
-                $('.ps-container.modal-body').perfectScrollbar('update');
+                this.scrollToTop();
             },
             (err) => {
                 var state = this.getStateFromStores();
@@ -129,6 +131,7 @@ export default class UserSettingsAppearance extends React.Component {
         const state = this.getStateFromStores();
         state.serverError = null;
         this.setState(state);
+        this.scrollToTop();
 
         Utils.applyTheme(state.theme);