Skip to content

feat(camera-startup): add Camera Startup widget to initialize camera …#7938

Open
WybrenKoelmans wants to merge 5 commits into
beyond-all-reason:masterfrom
WybrenKoelmans:camera-startup
Open

feat(camera-startup): add Camera Startup widget to initialize camera …#7938
WybrenKoelmans wants to merge 5 commits into
beyond-all-reason:masterfrom
WybrenKoelmans:camera-startup

Conversation

@WybrenKoelmans

@WybrenKoelmans WybrenKoelmans commented Jun 11, 2026

Copy link
Copy Markdown
Contributor
  • no animation/transition (instant at first frame)
  • fixed zoom level
  • doesn't touch camera rotation
  • positions start box in center
startup-cam.mp4

works with pretty extreme angles
https://github.com/user-attachments/assets/8ecf5e2e-1cee-41ee-a7a9-3c45f8741894

…at game start

- no animation/transition (instant at first frame)
- fixed zoom level
- doesn't touch camera rotation
- positions start box in center
@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Integration Test Results

15 tests   - 1   7 ✅  - 1   3s ⏱️ ±0s
 1 suites ±0   8 💤 ±0 
 1 files   ±0   0 ❌ ±0 

Results for commit ce749f7. ± Comparison against base commit 540b340.

This pull request removes 1 test.
ai_stai_factory_rect.lua

♻️ This comment has been updated with latest results.

@przystuj

Copy link
Copy Markdown
Contributor

Did you test it with different camera types? They use various camera params

@WybrenKoelmans

Copy link
Copy Markdown
Contributor Author

Yeah the only weird one is "first person" (points straight into the sky), but I don't expect anybody actually using that. If we want to be on the safe side, we can limit this widget to

  • overhead
  • rotatable overhead
  • spring

(we should even hide the other camera modes from the options imo, but thats another issue).

@WybrenKoelmans

Copy link
Copy Markdown
Contributor Author
0612.mp4

here you can see me testing camera modes, the "rotatable overhead" is slightly buggy, so I left it out as well in the end.

Comment thread luaui/Widgets/camera_startup.lua Outdated
Comment thread luaui/Widgets/camera_startup.lua
Comment thread luaui/Widgets/camera_startup.lua
Comment thread luaui/Widgets/camera_startup.lua
Comment thread luaui/Widgets/camera_startup.lua
Comment thread luaui/Widgets/camera_startup.lua
@WybrenKoelmans

Copy link
Copy Markdown
Contributor Author

thanks for the review guys, ready for another pass when you feel like it

@keithharvey keithharvey left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

looks good

@efrec efrec left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can self-remove after init also.

Comment thread luaui/Widgets/camera_startup.lua Outdated
Comment thread luaui/Widgets/camera_startup.lua Outdated
Comment thread luaui/Widgets/camera_startup.lua Outdated
Comment thread luaui/Widgets/camera_startup.lua Outdated
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.

5 participants