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

Refactor new method entry point to rwh #1041

Merged
merged 83 commits into from
Nov 7, 2023
Merged

Refactor new method entry point to rwh #1041

merged 83 commits into from
Nov 7, 2023

Conversation

wusyong
Copy link
Member

@wusyong wusyong commented Oct 22, 2023

RFC: tauri-apps/rfcs#12

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Docs
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change?

  • Yes
  • No

Checklist

  • This PR will resolve #___
  • A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
  • I have added a convincing reason for adding this feature, if necessary
  • It can be built on all targets and pass CI/CD.

Other information

Cargo.toml Outdated Show resolved Hide resolved
@amrbashir
Copy link
Member

amrbashir commented Oct 25, 2023

Here is what is left to do:

  • Fix Documentation and add missing ones
  • Cleanup examples and add new examples for winit, wgpu and some other projects
  • Figure out how to resize the child gtk in x11
  • Webview created directly in the passed X11, should resize when the X11 is resized? No

@wusyong
Copy link
Member Author

wusyong commented Oct 25, 2023

* [ ]  Cleanup examples and add new examples for winit, wgpu and some other projects

I think we should reduce the amount of example to only some basic ones and move others to integration test.
Let's do this in next isses/PRs.

* [ ]  Figure out how to resize the child gtk in x11

Can Windows auto resize? On macOS, it can auto resize. But I feel like we should just tell users to handle child position and size manually themselves.

@wusyong wusyong requested review from a team as code owners November 6, 2023 12:59
amrbashir added a commit to amrbashir/winit that referenced this pull request Nov 6, 2023
While working on tauri-apps/wry#1041 which allows adding the webview as a child window, I noticed that when setting the webview to be transparent, the portion of the parent behind it doesn't render properly and that was because of `WS_CLIPCHILDREN`
amrbashir added a commit to tauri-apps/tao that referenced this pull request Nov 6, 2023
While working on tauri-apps/wry#1041 which allows adding the webview as a child window, I noticed that when setting the webview to be transparent, the portion of the parent behind it doesn't render properly and that was because of `WS_CLIPCHILDREN`
wusyong pushed a commit to tauri-apps/tao that referenced this pull request Nov 7, 2023
While working on tauri-apps/wry#1041 which allows adding the webview as a child window, I noticed that when setting the webview to be transparent, the portion of the parent behind it doesn't render properly and that was because of `WS_CLIPCHILDREN`
@wusyong wusyong merged commit 783b142 into dev Nov 7, 2023
11 checks passed
@wusyong wusyong deleted the raw-handle branch November 7, 2023 11:30
@rhysd
Copy link
Contributor

rhysd commented Nov 9, 2023

@wusyong I think the overview description in README would need update as well :)

WRY connects the web engine on each platform and provides easy to use and unified interface to render WebView. It also re-exports TAO as a module for event loop and window creation.

https://github.com/tauri-apps/wry/blob/dev/README.md#overview

@wusyong
Copy link
Member Author

wusyong commented Nov 9, 2023

@rhysd Thanks for notifying! I've opened a PR to update it.
#1065

lucasfernog added a commit that referenced this pull request Nov 12, 2023
The android_binding macro fails to compile due to the `use $wry::prelude::*;` with `expected { or *` error.

Regression from #1041 which was not released yet so I didn't add a change file.
amrbashir pushed a commit that referenced this pull request Nov 12, 2023
* fix(android): properly `use $wry` on android_binding macro

The android_binding macro fails to compile due to the `use $wry::prelude::*;` with `expected { or *` error.

Regression from #1041 which was not released yet so I didn't add a change file.

* use {{ }}
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