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

feat: support for clip children on windows #16545

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nicholasc
Copy link

@nicholasc nicholasc commented Nov 28, 2024

Objective

Support the parametrization of the WS_CLIPCHILDREN style on Windows.

Fixes #16544

Solution

Added a window configuration in bevy_winit to control the usage of the WS_CLIPCHILDREN style.

Testing

  • Did you test these changes? If so, how?
    I did. I was able to create a Wry Webview with a transparent HTML document and was also able to see my Bevy scene behind the webview elements.

  • Are there any parts that need more testing?
    I don't believe so. I assume the option is extensively tested within winit itself.

  • How can other people (reviewers) test your changes? Is there anything specific they need to know?
    Test repositiory here. Bevy's path will need to be updated in the Cargo.toml

  • If relevant, what platforms did you test these changes on, and are there any important ones you can't test?
    This is a Windows specific issue. Should be tested accordingly.

Copy link
Contributor

Welcome, new contributor!

Please make sure you've read our contributing guide and we look forward to reviewing your pull request shortly ✨

@BenjaminBrienen BenjaminBrienen added C-Feature A new feature, making something new possible A-Windowing Platform-agnostic interface layer to run your app in D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Windowing Platform-agnostic interface layer to run your app in C-Feature A new feature, making something new possible D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add WS_CLIPCHILDREN support for Windows in bevy_winit
2 participants