Mattermost-server: Remove usage of jQuery from theme settings

Created on 31 Jul 2020  路  4Comments  路  Source: mattermost/mattermost-server

We've been working for some time to remove jQuery as a dependency of the Mattermost web app to reduce the size of the JS bundle in the web app and to simplify our code by using equivalent functions that are usable on all browsers that we support.

As part of this, we should remove the usage of jQuery from the following files in the web app:

  • components/user_settings/display/user_settings_theme/custom_theme_chooser.jsx
  • components/user_settings/display/user_settings_theme/premade_theme_chooser/premade_theme_chooser.jsx
  • utils/utils.jsx

In addition to removing the usage of jQuery, E2E tests using Cypress should be added to ensure that:

  • Sections properly animate open and closed when editing a custom theme in Account Settings > Display > Theme
  • Code themes are properly updated throughout the app when changing them from Account Settings > Display > Theme

If you have any questions, feel free to ask @hmhealey (or @harrison on our Mattermost community server).


If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please join our "Developers" community channel.

New contributors please see our Developer's Guide.

JIRA: https://mattermost.atlassian.net/browse/MM-27446

Medium Hacktoberfest Help Wanted PR Exists TecReactJS

All 4 comments

Hi, I can work on this.

Awesome, thanks! Let us know if you have any questions or anything.

This issue appears stale. and I鈥檓 adding the Up For Grabs label. If work is ongoing, please remove the label. If there is an existing PR, please also link it to the issue. Thanks!

I work on it now :)

Was this page helpful?
0 / 5 - 0 ratings