-
-
Notifications
You must be signed in to change notification settings - Fork 88
updates for Cogent Canvas #1528
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ng; key for cogent canvas which updates multiple sprites at a time and otherwise got into corrupted state with some sprites rendered before being all updated. also fixed key issue with path WriteGeom.
…simplifies most of the math involving them, especially on the canvas side.
…ul for many use-cases when viewing an SVG. also fix texteditor update on start of query-replace.
… system to update bound values for both Input and Change events, so it is always fresh. these are needed for proper undo handling in Canvas app.
…or text fields etc, so need this optional method to call when needed.
…t instead of having to search in the code for this every time..
… and body to widgets. also fixed saving of text elements in svg.
…lows the color picker to work. Add missing items to struct-tags docs. Add IconSize to styles: allows easy specification of icon size in a way that can be passed through chooser to the chooser menu -- minimal addition that elegantly solves this problem and makes canvas dashes and markers choosers actually functional.
…gets the proper type!
…rangely missing. doesn't break any tests or demo but might cause a bit more overhead..
…ng note to that effect.
…ay property for display:none checking
…y -- greatly simplifies tree icon size styling. switches needed some different logic but overall cleaner and controlled at the top-level of the switch instead of buried down inside the bits. overall gives the control at the top level where you want it!
… avoid duplicate names; Fixes #1007 Also exported functions needed in Canvas, and added DeleteSelected.
…vg.Group with no color! requires its own visible logic.
… are not exported (enum gen needs to respect that; doesn't currently). colors.Uniform returns nil if source color is nil, instead of a uniform wrapper -- verified this doesn't break any tests.
… recipe for writing text vs tspan
…rix2 "center" functions which were a wrong version of "about" logic which is now working correctly. also can get rid of the saving of geom stuff and just put gradients on nodes..
…ransforming nodes: use a simple BitCopyFrom of original node data instead of all the custom Read / Write geom, and store the node's own gradient points on the node itself, which gradient properties across new gradients and makes for much easier updating / editing of gradient geom. Got rid of all the faulty math32.Matrix2 center code which was incorrect version of About transforms. Everything is good except for our transform of the radius property on rasterx radius relative to the svg spec
…ale factors and then extract rotation and apply that to point during render, to put back into radius-based reference frame.
… just a couple examples for now)
…focus -- out-of-focus windows don't blink! minimizes any costs associated with this.
…there are many different colors and this is effectively a list of them)
…hat made me break down and write a shared TextCursor method..
kkoreilly
approved these changes
Jun 19, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Finally all good!
This was referenced Jun 19, 2025
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Some fixes needed to svg and other repos to make Cogent Canvas fully work.
Fixes #1314, fixes #1406