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

Improved day numbers with preview of what the day's content is #63

Open
jeroenheijmans opened this issue Jan 7, 2022 · 1 comment
Open
Labels
feature request Add a new feature. requires investigation Something still to be investigated

Comments

@jeroenheijmans
Copy link
Owner

It would be super nice if the various day numbers (especially in the "Podium per day" and "Stats per day" tables) would have some kind of (discoverable, accessible) tooltip or other extension to show what that day was about.

Especially looking at previous years, or even now in January looking back at past month, I struggle to know "what was that day again?". Being able to see "Day 21 'Dirac Dice'" and maybe a line or two fromt he page, would be nice.

Alternatively it could show, with a delay, a small popout window with the day in an iframe?

Need to be careful though as the days in the second table are already clickable button-like links. Not sure how the UX would work. In any case, leaving this trail/issue for myself for next year's edition.

@jeroenheijmans jeroenheijmans added the feature request Add a new feature. label Jan 7, 2022
@jeroenheijmans jeroenheijmans self-assigned this Dec 10, 2022
@jeroenheijmans
Copy link
Owner Author

Gave this some thought, but it's not trivial to do this. Some concerns:

  • Main concern: the feature should play nice with the AoC servers, and not cause malicious/overblown traffic. We can't just fire XHR's upon each hover of a day number.
  • Fragility: if we parse part of the HTML pages to find the title and first couple of paragraphs for a preview, that would rely on the html structure, which might change.
  • Hard-coding day titles (if even within what's allowed, since that content is copyrighted) doesn't work for the year when puzzles are coming out freshly.

I'm letting this issue sit until at least fall, not spending time on it until I at least know there will be a next edition of AoC at all.

@jeroenheijmans jeroenheijmans added the requires investigation Something still to be investigated label Jan 7, 2023
@jeroenheijmans jeroenheijmans removed their assignment Jan 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Add a new feature. requires investigation Something still to be investigated
Projects
None yet
Development

No branches or pull requests

1 participant