Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update viewer app #388

Merged
merged 2 commits into from
Jan 27, 2025
Merged

Update viewer app #388

merged 2 commits into from
Jan 27, 2025

Conversation

slimbuck
Copy link
Member

This PR:

  • store the viewer templates as text (instead of ts) and add a rollup plugin to load it at build time
    • this is simpler to develop (for now)
    • unfortunately we must rename the files with .template extension so they aren't picked up by postcss
  • viewer renders on demand (only when camera moves)
  • viewer has more accurate FOV handling compared to SuperSplat

Notes

  • It would be awesome if the template files could retain their extensions. If anyone knows how to accomplish this with rollup please submit a PR!
  • It will probably make sense to extract the viewer into a separate repo/package in future

@slimbuck slimbuck added the enhancement New feature label Jan 27, 2025
@slimbuck slimbuck requested a review from a team January 27, 2025 11:24
@slimbuck slimbuck self-assigned this Jan 27, 2025
@slimbuck slimbuck merged commit 8d76535 into playcanvas:main Jan 27, 2025
2 checks passed
@slimbuck slimbuck deleted the viewer-dev branch January 27, 2025 11:34
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants