Skip to content
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

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

baltpeter
Copy link
Member

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.

@baltpeter baltpeter marked this pull request as draft September 18, 2023 08:48
@cypress
Copy link

cypress bot commented Sep 18, 2023

Passing run #4977 ↗︎

0 80 8 0 Flakiness 0

Details:

squash me: Update Hugo (needed for findRESubmatch)
Project: datenanfragen/website Commit: 3b1b3f7c93
Status: Passed Duration: 04:16 💡
Started: Sep 18, 2023 8:53 AM Ended: Sep 18, 2023 8:58 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

Copy link
Member

@mal-tee mal-tee left a 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?

@baltpeter
Copy link
Member Author

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?

@mal-tee
Copy link
Member

mal-tee commented Dec 18, 2023

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.

@baltpeter
Copy link
Member Author

Do separate the slides from the background of the page. I guess a border or some other CSS styling could fix this as well.

But why? We don't separate "regular" images (e.g. in blog posts) from the background, either? :D

I'd also prefer a bolder line between slides.

Imo that looks silly but I don't care that much. What does @zner0L think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants