Skip to content

Fix sprite and picking examples#15803

Merged
alice-i-cecile merged 2 commits intobevyengine:mainfrom
Jondolf:fix-examples
Oct 10, 2024
Merged

Fix sprite and picking examples#15803
alice-i-cecile merged 2 commits intobevyengine:mainfrom
Jondolf:fix-examples

Conversation

@Jondolf
Copy link
Contributor

@Jondolf Jondolf commented Oct 9, 2024

Objective

Looks like #15489 broke some examples :) And there are some other issues as well.

Gabe's brother Gabe is tiny in the sprite_animation example:

kuva

Gabe is not running in the sprite_picking example, and (unrelated) is also very blurry: (note that the screenshot is a bit zoomed in)

kuva

Unrelated to sprites, the text in the simple_picking example is way too dark when hovered:

kuva

Solution

Both Gabes are now the correct size:

kuva

Gabe is crisp and running:

kuva

The text has better contrast:

kuva

@Jondolf Jondolf added D-Trivial Nice and easy! A great choice to get started with Bevy A-Rendering Drawing game state to the screen C-Examples An addition or correction to our examples X-Uncontroversial This work is generally agreed upon S-Needs-Review Needs reviewer attention (from anyone!) to move forward A-Picking Pointing at and selecting objects of all sorts labels Oct 9, 2024
@alice-i-cecile alice-i-cecile added this to the 0.15 milestone Oct 9, 2024
@alice-i-cecile alice-i-cecile added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Oct 9, 2024
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Oct 9, 2024
Merged via the queue into bevyengine:main with commit bd0c746 Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Picking Pointing at and selecting objects of all sorts A-Rendering Drawing game state to the screen C-Examples An addition or correction to our examples D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it X-Uncontroversial This work is generally agreed upon

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants