Closed camdendotlol closed 3 months ago
Name | Link |
---|---|
Latest commit | 4e56944e2666aea52c5cd3bdbe27bcb714718335 |
Latest deploy log | https://app.netlify.com/sites/avannotate-staging/deploys/66ab98b59db84c0009b72b6d |
Deploy Preview | https://deploy-preview-24--avannotate-staging.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Summary
This PR updates the
CompareEventsModal
andSingleEventModal
used by the rich text editor to require the user to specify the audiovisual file when the user is trying to embed a clip on an event with multiple AV files.The behavior is split into multiple cases (I'm not a fan of how complex this is from a dev standpoint - I worry that we might lose track of the different functionality branches in the future and this could become brittle):
file
property always existing instead of needing two branches of functionality for looking up filesScreenshots
Single Event
Event Comparison
(This modal overflowed on my laptop screen. I updated the styles so the buttom bar floats and the rest of the content is scrollable when necessary. This change will only be noticeable when a modal is too tall to fit on the screen.)