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

Create "grey" color aliases #128

Merged
merged 2 commits into from
Sep 25, 2018

Conversation

benjaminwil
Copy link
Collaborator

@benjaminwil benjaminwil commented Aug 5, 2018

This is my first time doing anything Python.

Some users prefer to spell "grey" with the non-American spelling. We should alias the "dark gray" and "light gray" colors to work with their spelling of "grey".

@benjaminwil benjaminwil changed the title Benjaminwil/color aliases Create color alias nested method for text color Aug 5, 2018
@modgethanc
Copy link
Contributor

this looks good to me! i just gave it a whirl by manually checking that alias in the client, and it's working. but, you make a good point with aliasing in general; it would be cool to expand on this in the future and have some default and user-editable color and verb aliasing.

@benjaminwil benjaminwil force-pushed the benjaminwil/color_aliases branch from 6b481af to aa55317 Compare September 1, 2018 20:11
Some users prefer to spell "grey" with the non-American spelling. We
should alias the "dark gray" and "light gray" colors to work with their
spelling of "grey".
@benjaminwil benjaminwil force-pushed the benjaminwil/color_aliases branch from aa55317 to 15ad3f4 Compare September 1, 2018 20:13
@benjaminwil benjaminwil changed the title Create color alias nested method for text color Create "grey" color aliases Sep 2, 2018
Copy link
Owner

@vilmibm vilmibm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!!

@vilmibm vilmibm merged commit fcb0cce into vilmibm:master Sep 25, 2018
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