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

Fixed: Default window is now "App" instead of "Bevy App" #9301

Merged
merged 1 commit into from
Jul 29, 2023

Conversation

floreal
Copy link
Contributor

@floreal floreal commented Jul 29, 2023

Objective

Fixes #9298 - Default window title leaks "bevy" context

Solution

I just replaced the literal string "Bevy App" with "App" in Window's Default trait implementation.

@github-actions
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 ✨

@Selene-Amanita Selene-Amanita 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 Jul 29, 2023
@Selene-Amanita Selene-Amanita added this to the 0.11.1 milestone Jul 29, 2023
Copy link
Member

@ickk ickk left a comment

Choose a reason for hiding this comment

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

Thanks!

@Selene-Amanita Selene-Amanita 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 Jul 29, 2023
@mockersf mockersf added this pull request to the merge queue Jul 29, 2023
Merged via the queue into bevyengine:main with commit 2515593 Jul 29, 2023
24 checks passed
@floreal floreal deleted the fix-default-window-title branch August 2, 2023 19:14
cart pushed a commit that referenced this pull request Aug 10, 2023
# Objective

Fixes #9298 - Default window title leaks "bevy" context

## Solution

I just replaced the literal string "Bevy App" with "App" in Window's
Default trait implementation.
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.

Default window title leaks "bevy" context
5 participants