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.

Sign up for GitHub

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

Jump to bottom

Add Drawable support for curves #109

Closed
3 tasks done
Tracked by #10
lucasstarsz opened this issue Aug 13, 2021 · 0 comments · Fixed by #203
Closed
3 tasks done
Tracked by #10

Add Drawable support for curves #109

lucasstarsz opened this issue Aug 13, 2021 · 0 comments · Fixed by #203
Assignees
@lucasstarsz
Milestone
FastJ Drawable Sy...

Comments

@lucasstarsz
Copy link
Member

lucasstarsz commented Aug 13, 2021 •
edited
Loading

As FastJ's drawable system grows, curves need to be supported by all polygons. The following tasks should be completed:

  • Add two sets of points for instances where curves/quads may be present (these sets of points mark out which indexes contain curve/quad points)
  • With the same principle as above, add curve/quad support to .psdf files
  • Add quad/curve-handling capabilities to DrawUtil's Path generation methods
@lucasstarsz lucasstarsz mentioned this issue Aug 13, 2021
Open
14 tasks
@lucasstarsz lucasstarsz changed the title Add support for curves Add Drawable support for curves Aug 13, 2021
@lucasstarsz lucasstarsz self-assigned this Aug 28, 2021
@lucasstarsz lucasstarsz added this to To do in FastJ 1.6.0 via automation Aug 28, 2021
@lucasstarsz lucasstarsz added this to the FastJ Drawable System milestone Aug 28, 2021
@lucasstarsz lucasstarsz removed this from To do in FastJ 1.6.0 Oct 12, 2021
lucasstarsz added a commit to lucasstarsz/FastJ that referenced this issue May 9, 2022
@lucasstarsz
(#10, fastjengine#109) Add curve support to Drawables
0d4ab51 
New Additions:
- Created Pair class to handle returning 2 values at once, for generating point meshes with curves.
lucasstarsz added a commit to lucasstarsz/FastJ that referenced this issue May 9, 2022
@lucasstarsz
(#10, fastjengine#109) curve path generation, .psdf support
a8bec73 
New Additions:
- Support for converting Path2D objects with curves and multiple subpaths to Pointf[] arrays (with optional `alternateIndexes` stored in paired `Point[]`)
- `DrawUtil#pointsOfPathWithAlt` specifically for getting both the points of the path and indicators of where curves/other path related functions are
- Support for writing/reading polygons with curves with .psdf files

Breaking Changes:
- `DrawUtil#pointsOfPath` and variants no longer throw exceptions on unclosed paths.

Bug Fixes:
- (minor) Fixed issue in `DrawUtil#createPath` where `altIndexes` was not checked for emptiness

Other Changes:
- Optimized usages of `Polygon2D#getPoints` to be less taxing for standard tasks
@lucasstarsz lucasstarsz mentioned this issue May 9, 2022
Merged
lucasstarsz added a commit that referenced this issue May 9, 2022
@lucasstarsz
(#10, #109) Add curve support to Drawables
db10125 
New Additions:
- Created Pair class to handle returning 2 values at once, for generating point meshes with curves.
lucasstarsz added a commit that referenced this issue May 9, 2022
@lucasstarsz
(#10, #109) curve path generation, .psdf support
949d3af 
New Additions:
- Support for converting Path2D objects with curves and multiple subpaths to Pointf[] arrays (with optional `alternateIndexes` stored in paired `Point[]`)
- `DrawUtil#pointsOfPathWithAlt` specifically for getting both the points of the path and indicators of where curves/other path related functions are
- Support for writing/reading polygons with curves with .psdf files

Breaking Changes:
- `DrawUtil#pointsOfPath` and variants no longer throw exceptions on unclosed paths.

Bug Fixes:
- (minor) Fixed issue in `DrawUtil#createPath` where `altIndexes` was not checked for emptiness

Other Changes:
- Optimized usages of `Polygon2D#getPoints` to be less taxing for standard tasks
lucasstarsz added a commit to lucasstarsz/FastJ that referenced this issue May 9, 2022
@lucasstarsz
(#10, fastjengine#109) add support for curves in DrawUtil#lengthOfPath
962d951
lucasstarsz added a commit that referenced this issue May 9, 2022
@lucasstarsz
Merge pull request #204 from lucasstarsz/main
6b6ce14 
(#10, #109) add support for curves in DrawUtil#lengthOfPath
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lucasstarsz lucasstarsz

Labels
None yet
Projects
None yet
Milestone
FastJ Drawable System
Development

Successfully merging a pull request may close this issue.

Curve support for Drawables lucasstarsz/FastJ
1 participant
@lucasstarsz