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

[First person] Add raycast usage #582

Merged
merged 11 commits into from
Dec 16, 2023
Merged

[First person] Add raycast usage #582

merged 11 commits into from
Dec 16, 2023

Conversation

D8H
Copy link
Contributor

@D8H D8H commented Oct 15, 2023

No description provided.

@D8H D8H added the 🔄 Example update An update for an existing example label Oct 15, 2023
@D8H D8H self-assigned this Oct 15, 2023
@github-actions
Copy link
Contributor

Preview the game(s) changed or added in this Pull Request


This is an automatic message displaying links to the games in this PR - double check the JSON in case of doubt.

@Bouh
Copy link
Contributor

Bouh commented Dec 13, 2023

This raycast condition should be an object-less condition i think. This is a bit confusing to bind the condition to the object.
It weird to select the object to test agaisnt the raycast then click on the raycast condition, then reading this test the camera against an object, that have been selected in the first step.
image

@D8H
Copy link
Contributor Author

D8H commented Dec 13, 2023

This raycast condition should be an object-less condition i think.

By "object-less" do you mean it should be in "Other conditions" or that it should not have any object parameter?

@Bouh
Copy link
Contributor

Bouh commented Dec 13, 2023 via email

@D8H
Copy link
Contributor Author

D8H commented Dec 13, 2023

Why this condition and not the others?
I mean they all do the same thing, the only difference is how the ray is described.

@D8H D8H force-pushed the first-person-raycast branch from 239f85a to 8756471 Compare December 16, 2023 22:47
@D8H D8H marked this pull request as ready for review December 16, 2023 22:58
@D8H D8H merged commit f716458 into main Dec 16, 2023
5 checks passed
@D8H D8H deleted the first-person-raycast branch December 16, 2023 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔄 Example update An update for an existing example
Projects
Status: Added to GDevelop
Development

Successfully merging this pull request may close these issues.

2 participants