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

added additional forms of extending DrRacket #372

Closed
wants to merge 4 commits into from

Conversation

spdegabrielle
Copy link
Member

@spdegabrielle spdegabrielle commented Apr 30, 2020

resolves issue #382

@spdegabrielle
Copy link
Member Author

@rfindler

I even fixed the typo!
image

@spdegabrielle
Copy link
Member Author

if you are happy please squash

@rfindler
Copy link
Member

rfindler commented Jun 3, 2020

I think @sorawee 's comments are good ones! :)

@spdegabrielle
Copy link
Member Author

spdegabrielle commented Jun 3, 2020 via email

@rfindler
Copy link
Member

rfindler commented Jun 3, 2020 via email

@spdegabrielle
Copy link
Member Author

New screenshot
image

@rfindler
Copy link
Member

rfindler commented Jun 4, 2020

Can we use the American spelling of "customization"? I also think it probably makes sense to write these in ascending order of complexity, so: keybindings, teachpacks, scripts, plugins, and then to say in the front para that it is in that order.

After that, I am happy with it. Thanks for this!

@spdegabrielle
Copy link
Member Author

image
Look good?

@rfindler
Copy link
Member

rfindler commented Jun 5, 2020

Looks great! Shall I merge?

@spdegabrielle
Copy link
Member Author

yes!

@rfindler rfindler closed this in 5636d44 Jun 5, 2020
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.

3 participants