Closed alanvww closed 3 months ago
Name | Link |
---|---|
Latest commit | 81b6757c327bccf8f2bf86b955dacff43dcc9104 |
Latest deploy log | https://app.netlify.com/sites/ml5-next-gen-docs/deploys/66a3d97e469a69000891d54d |
Deploy Preview | https://deploy-preview-152--ml5-next-gen-docs.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.
@alanvww Thank you so much! It looks great to me!! (Also with the edition by Dan)
@shiffman Thank you for the "calendar" idea. How about we maintain the banners in Fall 2024 and remove them during the winter break? (We keep approx. 5 months)
@MOQN @shiffman Thanks for the reviews! I have already committed the edited version. So if the preview deployment looks good, this PR is ready for merge :D
For the banner removal timeline, I think either way will work, starting with 3 months is the best bet. I will keep that in mind.
Hello @MOQN @QuinnHe , this is the test version of the banner with dismiss button and responsive for mobile. The style is temporary. Let me know what you think!