-
-
Notifications
You must be signed in to change notification settings - Fork 125
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Annotated presentation for Topio 2023 talk #1100
base: master
Are you sure you want to change the base?
Conversation
Passing run #4977 ↗︎
Details:
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great! But I am a bit torn on the design, why don't we wrap this in a slide show?
I adapted Simon's design for the most part. I do like having everything on one page for discoverability (scrolling is about as obvious as it gets). We do have a JS-free implementation of a carousel for the Honey act page but that is annoying to produce and… not great. Also, I'm not sure how it holds up a11y-wise. Why would you prefer a carousel, what benefits do you see? |
Do separate the slides from the background of the page. I guess a border or some other CSS styling could fix this as well. I'd also prefer a bolder line between slides. Maybe even introduce a new shade for the slide's caption? And I am once again proposing that we should bump the image quality. Imo these slides are compressed way too much. |
But why? We don't separate "regular" images (e.g. in blog posts) from the background, either? :D
Imo that looks silly but I don't care that much. What does @zner0L think? |
Inspired by https://simonwillison.net/2023/Aug/6/annotated-presentations/
This isn't quite done yet, @zner0L's new slides don't have text yet.