Closed gitriley closed 6 years ago
👍
On Sat, Jun 16, 2018 at 10:37 AM Riley Flynn notifications@github.com wrote:
I updated the presentation app so that the fullscreen carousel works. The main thing I had to do was update the html structure of the app so that the stacking order of the elements worked.
While I was there, I also updated the legend so that the legend container scrolls when there are too many elements (like 25+), rather than pushing down the height of the entire screen.
You can view, comment on, or merge this pull request online at:
https://github.com/LocalGround/localground/pull/196 Commit Summary
- fixed presentation fullscreen carousel and improved legend
File Changes
- M apps/templates/pages/presentation.html https://github.com/LocalGround/localground/pull/196/files#diff-0 (7)
- M static/apps/presentation/templates/legend-container.html https://github.com/LocalGround/localground/pull/196/files#diff-1 (3)
- M static/apps/presentation/views/layer-list-manager.js https://github.com/LocalGround/localground/pull/196/files#diff-2 (1)
- M static/css/map-presentation.css https://github.com/LocalGround/localground/pull/196/files#diff-3 (6)
- M static/lib/carousel/carousel.js https://github.com/LocalGround/localground/pull/196/files#diff-4 (1)
Patch Links:
- https://github.com/LocalGround/localground/pull/196.patch
- https://github.com/LocalGround/localground/pull/196.diff
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/LocalGround/localground/pull/196, or mute the thread https://github.com/notifications/unsubscribe-auth/ABKGw98oGFmU3pjKOL9upKLPFNDvjwCRks5t9UJ0gaJpZM4Uqjkm .
I updated the presentation app so that the fullscreen carousel works. The main thing I had to do was update the html structure of the app so that the stacking order of the elements worked.
While I was there, I also updated the legend so that the legend container scrolls when there are too many elements (like 25+), rather than pushing down the height of the entire screen.