Skip to content

Conversation

@andrewn
Copy link
Member

@andrewn andrewn commented Nov 25, 2019

This moves the logic for keeping the theme class on the body in sync with the redux store to a single place, App.jsx. This simplifies how non-IDEView pages can become theme-aware.

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • is from a uniquely-named feature branch and has been rebased on top of the latest master. (If I was asked to make more changes, I have made sure to rebase onto master then too)
  • is descriptively named and links to an issue number, i.e. Fixes #123

@andrewn andrewn mentioned this pull request Nov 25, 2019
13 tasks
@catarak catarak merged commit 7ec30dd into processing:master Nov 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants