Skip to content

Upgrade to Ink 1.1.0-alpha05#87

Merged
maxmmitchell merged 1 commit into
mainfrom
upgrade-ink-110alpha05
Jul 16, 2026
Merged

Upgrade to Ink 1.1.0-alpha05#87
maxmmitchell merged 1 commit into
mainfrom
upgrade-ink-110alpha05

Conversation

@maxmmitchell

Copy link
Copy Markdown
Contributor

Upgrade to the latest version of Ink as of this PR, and pull over recent updates to brush_family.proto from Ink's Github. Most applicable changes from this version increase for Cahier are:

  • Experimental release of partial stroke eraser. We can consider integrating this into the drawing notes portion of the app in addition to the full-stroke eraser already implemented.
  • Seems the overload of CanvasStrokeRenderer.create which takes forcePathRendering has been marked internal, and is only for cross-package usage within the Ink library -- we should probably be using the mesh renderer here anyway, so I just went ahead and changed that for DrawingCanvasViewModel.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the ink dependency to version 1.1.0-alpha05 and removes the forcePathRendering parameter from the CanvasStrokeRenderer.create call in DrawingCanvasViewModel.kt. Additionally, it introduces TilingTexture and StampingTexture messages within a new layer oneof field in brush_family.proto. I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@maxmmitchell
maxmmitchell marked this pull request as ready for review July 15, 2026 18:42

@cka-dev cka-dev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@maxmmitchell
maxmmitchell merged commit 758ef6a into main Jul 16, 2026
8 checks passed
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