Skip to content

[Merged by Bors] - Use circle for breakout example#5657

Closed
tkgalk wants to merge 3 commits intobevyengine:mainfrom
tkgalk:use-circle-for-breakout-example
Closed

[Merged by Bors] - Use circle for breakout example#5657
tkgalk wants to merge 3 commits intobevyengine:mainfrom
tkgalk:use-circle-for-breakout-example

Conversation

@tkgalk
Copy link
Contributor

@tkgalk tkgalk commented Aug 11, 2022

Objective

Solution

  • Uses the Mesh2D APIs to draw a circle. The collision still uses the AABB algorithm, but it seems to be working fine, and I haven't seen any odd looking cases.

@alice-i-cecile alice-i-cecile added A-Rendering Drawing game state to the screen C-Examples An addition or correction to our examples labels Aug 11, 2022
@tkgalk
Copy link
Contributor Author

tkgalk commented Aug 11, 2022

Seems that I made the branch from latest instead of main. I'll rebase when I get up tomorrow (around 8-10h from now). 👍🏻

@tkgalk
Copy link
Contributor Author

tkgalk commented Aug 12, 2022

Oh, it is rebased on main. It seems the CI issue is unrelated.

@NiklasEi NiklasEi 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 Aug 13, 2022
@cart
Copy link
Member

cart commented Aug 16, 2022

bors r+

@cart
Copy link
Member

cart commented Aug 16, 2022

bors retry

bors bot pushed a commit that referenced this pull request Aug 16, 2022
# Objective

- Replace the square with a circle in the breakout example.
- Fixes #4324, adopted from #4682 by @shaderduck.

## Solution
- Uses the Mesh2D APIs to draw a circle. The collision still uses the AABB algorithm, but it seems to be working fine, and I haven't seen any odd looking cases.
@bors
Copy link
Contributor

bors bot commented Aug 16, 2022

@bors bors bot changed the title Use circle for breakout example [Merged by Bors] - Use circle for breakout example Aug 16, 2022
@bors bors bot closed this Aug 16, 2022
@tkgalk tkgalk deleted the use-circle-for-breakout-example branch August 18, 2022 14:55
maccesch pushed a commit to Synphonyte/bevy that referenced this pull request Sep 28, 2022
# Objective

- Replace the square with a circle in the breakout example.
- Fixes bevyengine#4324, adopted from bevyengine#4682 by @shaderduck.

## Solution
- Uses the Mesh2D APIs to draw a circle. The collision still uses the AABB algorithm, but it seems to be working fine, and I haven't seen any odd looking cases.
james7132 pushed a commit to james7132/bevy that referenced this pull request Oct 28, 2022
# Objective

- Replace the square with a circle in the breakout example.
- Fixes bevyengine#4324, adopted from bevyengine#4682 by @shaderduck.

## Solution
- Uses the Mesh2D APIs to draw a circle. The collision still uses the AABB algorithm, but it seems to be working fine, and I haven't seen any odd looking cases.
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
# Objective

- Replace the square with a circle in the breakout example.
- Fixes bevyengine#4324, adopted from bevyengine#4682 by @shaderduck.

## Solution
- Uses the Mesh2D APIs to draw a circle. The collision still uses the AABB algorithm, but it seems to be working fine, and I haven't seen any odd looking cases.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Rendering Drawing game state to the screen C-Examples An addition or correction to our examples 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.

Use a circle for the ball in Breakout

6 participants