Skip to content

Extract skiko platform code into separate module - #1292

Open
Hubert Błaszczyk (hub-bla) wants to merge 8 commits into
masterfrom
hub.bla/skiko-rendering-module
Open

Hubert Błaszczyk (hub-bla) wants to merge 8 commits into
masterfrom
hub.bla/skiko-rendering-module

Conversation

@hub-bla

@hub-bla Hubert Błaszczyk (hub-bla) commented Sep 7, 2026 •

Copy link
Copy Markdown
Contributor

This branch introduces a new skiko-rendering module that separates platform rendering integration from the core skiko library.

The new module owns the higher-level rendering infrastructure, including SkiaLayer, frame dispatching, render delegates, AWT/Swing integration, platform renderers, ANGLE support.

Fixes SKIKO-1189

@hub-bla
Hubert Błaszczyk (hub-bla) force-pushed the hub.bla/skiko-rendering-module branch 3 times, most recently from ca727bc to 660f7ae Compare September 7, 2026 12:10
@hub-bla
Hubert Błaszczyk (hub-bla) force-pushed the hub.bla/skiko-rendering-module branch 2 times, most recently from 098c342 to 03108b0 Compare September 7, 2026 13:07
Base automatically changed from hub.bla/decouple-ganesh-from-platform-redrawers to master September 9, 2026 09:36
@hub-bla
Hubert Błaszczyk (hub-bla) force-pushed the hub.bla/skiko-rendering-module branch 5 times, most recently from 94198bd to 9218f96 Compare September 11, 2026 11:15
@hub-bla
Hubert Błaszczyk (hub-bla) marked this pull request as ready for review September 14, 2026 08:54
Comment thread skiko/settings.gradle.kts
Comment thread skiko/settings.gradle.kts Outdated
@hub-bla Hubert Błaszczyk (hub-bla) changed the title Extract skiko rendering code into separate module Extract skiko platform code into separate module Sep 21, 2026

@igordmn Igor Demin (igordmn) 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.

Only skiko/README.md needs to be updated

@igordmn

Igor Demin (igordmn) commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Please adopt the new changes from master

Comment thread skiko/skiko-platform/src/jvmMain/kotlin/org/jetbrains/skiko/Setup.kt Outdated
implementation("org.jetbrains.kotlinx:kotlinx-coroutines-swing:1.5.0")
implementation("org.jetbrains.kotlinx:kotlinx-coroutines-test:1.5.0")
implementation("org.jetbrains.skiko:skiko-awt-runtime-$target:$version")
implementation("org.jetbrains.skiko:skiko-platform-awt-runtime-$target:$version")

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.

The issue is described in #1301 (comment), should be fixed or discussed before merging.

This branch has not been deployed

No deployments
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