Skip to content

Conversation

@HeikoKlare
Copy link

This is currently for demonstration purpose and not to be merged.

With this change, the SkijaGC is extended by the capability of rendering directly into the buffer of a GLCanvas, and this functionality is applied to the Button.

This change drastically improves performance as the bad-performing copy operation between Skia buffer and native buffer is avoided. Initial performance tests show a severe speedup of > 100 (i.e. 10.000%).

@HeikoKlare HeikoKlare marked this pull request as draft November 15, 2024 19:28
DenisUngemach and others added 29 commits November 16, 2024 11:02
Signed-off-by: Denis Ungemach <[email protected]>
Signed-off-by: Denis Ungemach <[email protected]>
Signed-off-by: Denis Ungemach <[email protected]>
Signed-off-by: Denis Ungemach <[email protected]>
Signed-off-by: Denis Ungemach <[email protected]>
That way it will be rendered in Linux (as a black square so it still
doesn't really work)
Signed-off-by: Denis Ungemach <[email protected]>
If you want to use it change super class of `Text` from `CText` to
`CSimpleText`.
All 3 classes were copy-pasted from this article:
https://www.eclipse.org/articles/Article-Writing%20Your%20Own%20Widget/Writing%20Your%20Own%20Widget.htm

Only the absolutely necessary changes were made so the code compiles.
On MacOS, it still uses the own rendering by inheriting from `CCaret`
Signed-off-by: Denis Ungemach <[email protected]>
The implementation is missing though.
Its implementation is empty
@HeikoKlare HeikoKlare changed the base branch from master to skija November 16, 2024 10:07
@HeikoKlare HeikoKlare mentioned this pull request Dec 4, 2024
38 tasks
@fedejeanne fedejeanne force-pushed the skija branch 2 times, most recently from 659fde0 to 75c432c Compare January 28, 2025 12:17
@HeikoKlare HeikoKlare force-pushed the master branch 4 times, most recently from a9f24e5 to d2109d6 Compare March 4, 2025 16:50
@tmssngr tmssngr force-pushed the master branch 2 times, most recently from 3d97fac to aa02556 Compare June 23, 2025 08:42
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.

5 participants