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

[Merged by Bors] - Expose winit always_on_top #6527

Closed
wants to merge 8 commits into from

Conversation

laundmo
Copy link
Contributor

@laundmo laundmo commented Nov 9, 2022

Objective

I needed a window which is always on top, to create a overlay app.

Solution

expose the always_on_top property of winit in bevy's WindowDescriptor as a boolean flag


Changelog

Added

  • add WindowDescriptor.always_on_top which configures a window to stay on top.

@alice-i-cecile alice-i-cecile added A-Windowing Platform-agnostic interface layer to run your app in C-Usability A targeted quality-of-life change that makes Bevy easier to use labels Nov 9, 2022
Copy link
Member

@alice-i-cecile alice-i-cecile left a comment

Choose a reason for hiding this comment

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

Doc nit, then LGTM. I'm surprised this is your first PR!

@rparrett
Copy link
Contributor

rparrett commented Nov 9, 2022

Support for this should probably be also added to Window and WindowCommand so it can be changed at runtime.

I tried this out and it's pretty simple to add and works as expected (on MacOS at least)

Feel free to use this: rparrett@fd2a706 (but add missing docs and whatnot)

@DJMcNab
Copy link
Member

DJMcNab commented Nov 9, 2022

I'm not sure this needs its own example.

I think there's an existing window_properties example?

@laundmo
Copy link
Contributor Author

laundmo commented Nov 9, 2022

Feel free to use this: rparrett@fd2a706 (but add missing docs and whatnot)

will do

@rparrett
Copy link
Contributor

rparrett commented Nov 9, 2022

I'm not sure this needs its own example.

I think there's an existing window_properties example?

Once support for changing at runtime is in, this could definitely be worked into the window_settings example.

@Weibye Weibye added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Nov 11, 2022
@cart
Copy link
Member

cart commented Nov 14, 2022

bors r+

bors bot pushed a commit that referenced this pull request Nov 14, 2022
# Objective

I needed a window which is always on top, to create a overlay app.

## Solution

expose the `always_on_top` property of winit in bevy's `WindowDescriptor` as a boolean flag

---

## Changelog

### Added
- add `WindowDescriptor.always_on_top` which configures a window to stay on top.
@bors
Copy link
Contributor

bors bot commented Nov 14, 2022

@bors bors bot changed the title Expose winit always_on_top [Merged by Bors] - Expose winit always_on_top Nov 14, 2022
@bors bors bot closed this Nov 14, 2022
taiyoungjang pushed a commit to taiyoungjang/bevy that referenced this pull request Dec 15, 2022
# Objective

I needed a window which is always on top, to create a overlay app.

## Solution

expose the `always_on_top` property of winit in bevy's `WindowDescriptor` as a boolean flag

---

## Changelog

### Added
- add `WindowDescriptor.always_on_top` which configures a window to stay on top.
alradish pushed a commit to alradish/bevy that referenced this pull request Jan 22, 2023
# Objective

I needed a window which is always on top, to create a overlay app.

## Solution

expose the `always_on_top` property of winit in bevy's `WindowDescriptor` as a boolean flag

---

## Changelog

### Added
- add `WindowDescriptor.always_on_top` which configures a window to stay on top.
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
# Objective

I needed a window which is always on top, to create a overlay app.

## Solution

expose the `always_on_top` property of winit in bevy's `WindowDescriptor` as a boolean flag

---

## Changelog

### Added
- add `WindowDescriptor.always_on_top` which configures a window to stay on top.
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-Usability A targeted quality-of-life change that makes Bevy easier to use S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants