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 example with rectangular canvas (for testing) #87

Merged
merged 1 commit into from
Jan 16, 2024

Conversation

nikochiko
Copy link
Collaborator

A circle in a rectangular canvas should not become an ellipse.

A circle in a rectangular canvas should not become an ellipse.
@nikochiko nikochiko merged commit 6a9209d into main Jan 16, 2024
0 of 3 checks passed
@nikochiko nikochiko deleted the rectangular-canvas branch January 16, 2024 21:52
FayCarsons pushed a commit to FayCarsons/ocaml-joy that referenced this pull request Jan 19, 2024
A circle in a rectangular canvas should not become an ellipse.
FayCarsons pushed a commit to FayCarsons/ocaml-joy that referenced this pull request Jan 26, 2024
A circle in a rectangular canvas should not become an ellipse.
FayCarsons pushed a commit to FayCarsons/ocaml-joy that referenced this pull request Feb 16, 2024
A circle in a rectangular canvas should not become an ellipse.
nikochiko added a commit that referenced this pull request Feb 16, 2024
* first draft color system

* rewriting examples

* dune fmt

* add proposal for color transforms

* Add example with rectangular canvas (for testing) (#87)

A circle in a rectangular canvas should not become an ellipse.

* Fix scaling factor in scale transform (#88)

* Fix scaling factor: shouldn't be sqrt

* Format with dune

* Add donut-with-scaling example

* more merging ??

* merging w/ main branch

* added stroke and fill fields, 'with' fns to add them to shapes, and map fns for applying fns to those fields

* applying suggestions and adding map_color fns

* remove map_color

* change docstring phrasing, fix .mli files

* fixing merge issues

---------

Co-authored-by: Kaustubh Maske Patil <[email protected]>
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.

1 participant