-
Notifications
You must be signed in to change notification settings - Fork 128
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
Fixes Disabling of Markdown buttton on using shortcut keys #465
Conversation
The extra spelling changes were also included as in PR #426 changes were directly made to dist files and these spelling mistakes were in node modules. |
@jywarren @VladimirMikulic Please review the changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
@jywarren @emilyashley Can you please merge this!! |
Oh this makes a ton of sense. Thanks a lot! |
I just synced the branch so this should be good to merge once it passes. I'll check back in! |
This is so odd. I'll close and reopen to try to retrigger the tests. If this happens again, try that too! |
And if it keeps happening, it might need a clean rebase... just an idea! |
Aha! Done!!! 🎉 |
Fixes #427
before changes pressing CTRL + M disabled the Markdown button. Now, It has been fixed.
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
grunt jasmine
fixes #0000
-style reference to original issue #@publiclab/reviewers
for help, in a comment belowIf tests do fail, click on the red
X
to learn why by reading the logs.Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software
We have a loose schedule of reviewing and pulling in changes every Tuesday and Friday, and publishing changes on Fridays.
Thanks!