Skip to content

[examples] shapes_collision_ellipses#5722

Open
Monjaris wants to merge 2 commits intoraysan5:masterfrom
Monjaris:feat/shapes-collision-ellipses
Open

[examples] shapes_collision_ellipses#5722
Monjaris wants to merge 2 commits intoraysan5:masterfrom
Monjaris:feat/shapes-collision-ellipses

Conversation

@Monjaris
Copy link
Copy Markdown

@Monjaris Monjaris commented Apr 4, 2026

No description provided.

{
// Initialization
//--------------------------------------------------------------------------------------
const int screenWidth = 1200;
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Examples should be 800x450, please could you review it

//----------------------------------------------------------------------------------
BeginDrawing();

ClearBackground(bgColor);
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please, could you use ClearBackground(RAYWHITE);, it's the default useed in most examples


ClearBackground(bgColor);

DrawEllipse((int)ellipseACenter.x, (int)ellipseACenter.y,
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please, could those draws be a single line, thanks

@raysan5
Copy link
Copy Markdown
Owner

raysan5 commented Apr 5, 2026

@Monjaris thanks for the new example, I added some comments to review.

@raysan5 raysan5 changed the title Added shapes_collision_ellipses example [examples] shapes_collision_ellipses Apr 5, 2026
@Monjaris
Copy link
Copy Markdown
Author

Sorry for late response. i fixed changed now is it applicable and mergable?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants