New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Save options as they're changed; Update borders immediately #160

Merged
merged 4 commits into from May 10, 2018

Conversation

1 participant
@matatk
Owner

matatk commented May 10, 2018

This removes the need for the user to press the “Save” button to save options (and removes the button).

The borders will be redrawn/removed to reflect changing settings.

Closes #22

matatk added some commits May 9, 2018

Save options on change
* Save options on change, rather than having to press the Save button.
* Fix a bug with the detection of unused messages.
* Remove now-unused messages.
Updating border if needed
* Update (redraw or remove) the border when needed.
* Big clean-ups and logic improvements in content.focusing.js.
* Remove debugging info from options.js.

@matatk matatk changed the title from Options without saving to Save options as they're changed; Update borders immediately May 10, 2018

@matatk matatk merged commit d8664a5 into master May 10, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@matatk matatk deleted the options-without-saving branch May 10, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment