Add animated dice rolls #511
Conversation
|
Visit the preview URL for this PR (updated for commit 860170f): https://iron-fellowship-dev--pr511-feat-animated-dice-xmxaypdy.web.app (expires Sun, 10 Nov 2024 04:40:30 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 4789204539639faf2e056a0ca1afe6984016b5d7 |
|
Visit the preview URL for this PR (updated for commit 860170f): https://crew-link-dev--pr511-feat-animated-dice-h4w5ggzj.web.app (expires Sun, 10 Nov 2024 04:39:20 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: ac5b7c2b217e7297177075d0f9d5001772953ac5 |
src/api-calls/user/_user.type.ts
Outdated
| @@ -8,4 +8,5 @@ export interface UserDocument { | |||
| layout?: { | |||
| referenceSidebarLocation?: ReferenceSidebarLocation; | |||
| }; | |||
| hideDice?: boolean; | |||
There was a problem hiding this comment.
Nit - this could be a little more specific.
Perhaps show3dDice if the feature is opt-in or hide3dDice if opt out?
src/styles.css
Outdated
| #dice-box canvas { | ||
| position: fixed; | ||
| pointer-events: none; | ||
| z-index: 100; |
There was a problem hiding this comment.
One minor issue on mobile: the dice sometimes land underneath the floating top bar. I think I'd rather have them appear overt top, so some z-index changes might need to be made.
Add configurable animated dice rolls using https://www.npmjs.com/package/@3d-dice/dice-box