UWRealityLab / xrcapstone22wi-team8

XR whiteboard, maybe a little more
https://UWRealityLab.github.io/xrcapstone22wi-team8
3 stars 3 forks source link

Implement dropdown for adding transferred contents #20

Closed ApolloZhu closed 2 years ago

ApolloZhu commented 2 years ago

Important

This merges into mvp instead of master, unless mvp has already been merged into master

Changelog

Known Issues

ApolloZhu commented 2 years ago

I've requested @ebanisadr's review since you'll need to implement displaying images on top of changes introduced in this PR. Let me know if I should alter anything for your needs that you anticipate.

ebanisadr commented 2 years ago

A problem I'm having is that generated objects that collide with each other get a velocity, and since they're not affected by gravity they tend to float away; see video.

https://user-images.githubusercontent.com/24218339/153061673-469a76e8-0662-4bba-ac49-d12481ed5b8a.mp4

ApolloZhu commented 2 years ago

@ebanisadr that is a problem with the MVP branch, which this PR doesn't change. unity-firebase branch should have the dropdown menu for selecting files and I'm not seeing that in your recording. Can you confirm you are building the correct branch?

ApolloZhu commented 2 years ago

@KH-Kyle is it safe to remove the RigidBody from primitive prefabs now? I believe that will fix the floating away issue (and you can commit directly to MVP after testing

ebanisadr commented 2 years ago

Good point, thought this was the PR to merge mvp into master.

KH-Kyle commented 2 years ago

@KH-Kyle is it safe to remove the RigidBody from primitive prefabs now? I believe that will fix the floating away issue (and you can commit directly to MVP after testing

Yeah. We changed them during yesterday's meeting but it didn't update in the repo. I just update the mvp branch.