Skip to content
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

CfW: let ComposeWindow accept a custom canvas id (html canvas element id) #626

Merged
merged 2 commits into from
Jul 7, 2023

Conversation

eymar
Copy link
Member

@eymar eymar commented Jul 7, 2023

This PR "upstreams" the changes from wasm-main to jb-main:

The canvas element id was hardcoded. Now it's possible to set any canvas id.

Also, the canvas will automatically set its size to the browser window and it will resize automicatically. It's the default behaviour, that can be changed by a user.

… id)

It upstreams the change from wasm-main branch to jb-main branch
@eymar eymar requested a review from dima-avdeev-jb July 7, 2023 09:14
@eymar eymar merged commit f05c388 into jb-main Jul 7, 2023
@eymar eymar deleted the cfw_customize_compose_window_canvasid branch July 7, 2023 10:41
Walingar pushed a commit to Walingar/androidx that referenced this pull request Jul 11, 2023
… id) (JetBrains#626)

* CfW: let ComposeWindow accept a custom canvas id (html canvas element id)

It upstreams the change from wasm-main branch to jb-main branch

* PR review fixed
mazunin-v-jb pushed a commit that referenced this pull request Jul 11, 2023
… id) (#626)

* CfW: let ComposeWindow accept a custom canvas id (html canvas element id)

It upstreams the change from wasm-main branch to jb-main branch

* PR review fixed
eymar pushed a commit that referenced this pull request Dec 18, 2023
## Proposed Changes

  - Update Revved up by Develocity badge

## Testing

Test: Describe how you tested your changes. Note that this line (with `Test:`) is required, your PR will not build without it!

## Issues Fixed

Fixes: [Optional] The bug on [https://issuetracker.google.com](https://issuetracker.google.com) being fixed

This is an imported pull request from androidx#626.

Resolves #626
Github-Pr-Head-Sha: 8b2718b
GitOrigin-RevId: f5ff0ef
Change-Id: I4cd12870cb403e858f249054bbc7500a45ae627a
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