Skip to content
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

add tools menu items, and fix a few shortcuts, first try #564

Merged
merged 1 commit into from
Jun 3, 2019

Conversation

truthgoddess
Copy link
Contributor

Should close #448 if merged.

@stevekrouse I suspect you might have questions and comments on some things. Happy to go back and apply your suggestions.

@stevekrouse
Copy link
Owner

[This is an automated integration to preview this pull request's changes to the website.]

https://cdn.rawgit.com/stevekrouse/WoofJS/c5e7c2f24b499f801f6851d454fdca4571991570/index.html

@stevekrouse
Copy link
Owner

The code looks perfect 👏

I’m currently unable to do a thorough test of all of the features, so just make sure that everything is working properly, and then feel free to merge.

@truthgoddess
Copy link
Contributor Author

Thanks Steve. I'm happy to hear that. It doesn't look that complex, but it took me a bit of thinking to get that copy, cut, and paste working. :) @madduccino Can you test on your end the new menu items, that ctrl b with nothing selected works, and that the clean code grey out looks more significant?

@truthgoddess truthgoddess merged commit 17034d0 into master Jun 3, 2019
@truthgoddess truthgoddess deleted the 8-new-tools branch June 4, 2019 14:28
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.

more keyboard shortcuts in tools menu
2 participants