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 514 rgb() support #254

Merged
merged 22 commits into from
Mar 19, 2021
Merged

Add 514 rgb() support #254

merged 22 commits into from
Mar 19, 2021

Conversation

ZeWaka
Copy link
Contributor

@ZeWaka ZeWaka commented Mar 12, 2021

Allows for rgb() with arguments of form [r,g,b|h,s,[v|l|y]],(a),(space)

Also allows the full names of the arguments.
Technically, BYOND supports just the first letter so halifax works for hue but fuck that.

Also now lints to see if the values are in the valid ranges.

image
image
image
image

@ZeWaka ZeWaka changed the title we can now have 5 args with rgb(r,g,b,a,space) full 514 rgb() support Mar 15, 2021
@ZeWaka ZeWaka requested a review from SpaceManiac March 15, 2021 05:16
src/dreammaker/constants.rs Outdated Show resolved Hide resolved
@SpaceManiac SpaceManiac changed the title full 514 rgb() support Add 514 rgb() support Mar 19, 2021
Copy link
Owner

@SpaceManiac SpaceManiac left a comment

Choose a reason for hiding this comment

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

Accepting this functionality as-is + some non-functional code cleanup. Looking in to extra tests for weirdo cases.

@SpaceManiac SpaceManiac merged commit 7e3731e into SpaceManiac:master Mar 19, 2021
@ZeWaka ZeWaka deleted the rgbstuff branch March 19, 2021 04:59
j-awn pushed a commit to j-awn/SpacemanDMM that referenced this pull request Apr 1, 2021
Allows for `rgb()` with arguments of form 
`[r,g,b|h,s,[v|l|y]],(a),(space)`

Also allows the full names of the arguments. Technically, BYOND 
supports just the first letter so `halifax` works for `hue` but fuck 
that.

Also now lints to see if the values are in the valid ranges.

Co-authored-by: Tad Hardesty <tad@platymuus.com>
@SpaceManiac SpaceManiac added this to the suite v1.7 milestone May 19, 2021
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.

2 participants