Skip to content

Comments

Make WebElementView API accessible in webMain source set#2614

Merged
eymar merged 4 commits intojb-mainfrom
ok/use_webMain
Dec 3, 2025
Merged

Make WebElementView API accessible in webMain source set#2614
eymar merged 4 commits intojb-mainfrom
ok/use_webMain

Conversation

@eymar
Copy link
Member

@eymar eymar commented Dec 2, 2025

Without this change WebElementView can be used only separately in jsMain and in wasmJsMain.

Fixes https://youtrack.jetbrains.com/issue/CMP-9146

  • Refactoring of web related source sets: moved most of the things from "fake" webW3C source set to webMain
  • Removed some web expect/actuals, leaving just one implementation for both targets
  • Had to add some new expect/actuals due to kotlin-js interop difference in the targets
  • Bumped LV and api version to 2_2 to fix the metadata compilation for web (using "new" js interop API)

Testing

Existing tests

Release Notes

Fixes - Web

  • WebElementView is now available in webMain source set
  • Deprecated CanvasBasedWindow has been deleted

Migration Notes - Multiple Platforms

  • The project has migrated to Kotlin language version and API version 2.2. The projects depending on Compose Multiplatform need to use the same version or newer.

@eymar eymar changed the title Ok/use web main Make WebElementView API accessible in webMain source set Dec 2, 2025
*/

package androidx.compose.ui.window
package androidx.compose.ui.platform
Copy link
Member Author

Choose a reason for hiding this comment

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

It's not a rename/move of a file. GitHub is just confused.

}

class JsPlatformClipboard : Clipboard {
private class JsPlatformClipboard : Clipboard {
Copy link
Member Author

Choose a reason for hiding this comment

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

It was public by mistake.

}

class WasmPlatformClipboard : Clipboard {
private class WasmPlatformClipboard : Clipboard {
Copy link
Member Author

Choose a reason for hiding this comment

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

It was public by mistake.

@eymar eymar requested a review from MatkovIvan December 2, 2025 15:59
Copy link
Member

@MatkovIvan MatkovIvan left a comment

Choose a reason for hiding this comment

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

Such changes require two approvals

@igordmn igordmn removed their request for review December 3, 2025 14:13
@igordmn
Copy link
Collaborator

igordmn commented Dec 3, 2025

Removed myself from review, as Shagen approved, and Ivan review was requested

@eymar eymar merged commit af3ce5e into jb-main Dec 3, 2025
16 checks passed
@eymar eymar deleted the ok/use_webMain branch December 3, 2025 15:45
pjBooms added a commit to JetBrains/compose-multiplatform that referenced this pull request Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants