Skip to content

Camera composition #19706

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

Closed
wants to merge 22 commits into from
Closed

Camera composition #19706

wants to merge 22 commits into from

Conversation

ecoskey
Copy link
Contributor

@ecoskey ecoskey commented Jun 17, 2025

Putting this up for discussion's sake

am too lazy to write a description rn.

@alice-i-cecile alice-i-cecile added A-Rendering Drawing game state to the screen S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged labels Jun 17, 2025
@alice-i-cecile alice-i-cecile added the C-Feature A new feature, making something new possible label Jun 17, 2025
ecoskey added 3 commits June 17, 2025 13:33
rename ViewTarget to MainCameraTextures

WIP

WIP

silenced most errors, left some todos

next up: crate hierarchy!
everything is broken oh god

WIP

WIP

WIP

WIP
WIP

WIP

fix errors

most are dummy fixes, just so I could get IDE messages out of my way

comments and todos abound, will be fixed next :)

WIP

WIP

WIP

WIP

WIP

WIP

add todo msg

WIP

WIP

update comment
@ecoskey ecoskey force-pushed the camera_composition branch from f2c6ca7 to 0139c77 Compare June 17, 2025 20:33
@ecoskey
Copy link
Contributor Author

ecoskey commented Jun 19, 2025

closing, I don't think this current iteration will get to a state worth merging and was mostly for discussion visibility

@ecoskey ecoskey closed this Jun 19, 2025
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-Feature A new feature, making something new possible S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants