Closed jayvarner closed 2 years ago
Name | Link |
---|---|
Latest commit | 906d674a0fa41df1fb75c3caaa156990b048dd9b |
Latest deploy log | https://app.netlify.com/sites/databydesign/deploys/6336f0c957dd9e000936a7b6 |
Deploy Preview | https://deploy-preview-93--databydesign.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 settings.
Seems like the padding changed on the scrollytell.
Before:
After:
I guess it sort of depends on how tall your window is. It is higher in the initial state than it was before. I did that to make room for the legend that is added between the 2nd and 3rd group. The div for the legend is just added, so there is not a great way to animate/ease the transition. Here is where I added a placeholder for now.
All that said, there is wonkiness on short windows. I'll take another look.
My last commit adds ensures there is some top spacing on short windows. The addition of the legend still causes some shiftiness on windows < 1000px. There is another task on #58 to "Add Ease in Animations." I can address it when I work on that.
This might impact other Scrollytell visualizations that I missed. Let me know and I'll amend the PR. This fixes #80 and likely affects #77 and #69.