Skip to content

Conversation

@riastrad
Copy link
Owner

@riastrad riastrad commented Nov 6, 2025

Closes #274

This updates the currently reading sections of the website to trigger a popover with some additional information. Rather than just have them be jumplinks to external sites.

Still to do:

  • Fix spacing issues with popover divs
  • Fix popover styling
  • Add a ::backdrop to popovers

Won't do:

  • Incorporate popover solution on bookshelf's 'currently reading' section

@riastrad
Copy link
Owner Author

riastrad commented Nov 6, 2025

Here's the current state of the home page. I'm happy with it so far. Very happy that I can get to this point without adding new javascript to the home page.

image

Only issues remaining for home page are some spacing issues caused by the parser inserting unexpected <p> elements. Ideally I'd find a way to prevent those elements from ending up in the final build...

@riastrad
Copy link
Owner Author

riastrad commented Nov 6, 2025

💡 idea for a blogpost: <progress> is a nightmare

@riastrad
Copy link
Owner Author

riastrad commented Nov 6, 2025

Style is in a good place so I'm going to stop fiddling. Need to fix the spacing before this can merge though.

Not entirely convinced I should wrestle this onto the bookshelf page yet...

@riastrad
Copy link
Owner Author

riastrad commented Nov 6, 2025

Pretty sure I've nailed this down for the home page:

desktop
image

mobile screens
image

@riastrad
Copy link
Owner Author

riastrad commented Nov 6, 2025

I'm going to run with this and skip the bookshelf updates for now. Can revisit in future if it feels like they're needed to feel consistent.

@riastrad riastrad marked this pull request as ready for review November 6, 2025 06:43
@riastrad riastrad merged commit c1139cf into main Nov 6, 2025
2 checks passed
@riastrad riastrad deleted the reading-progress branch November 6, 2025 06:46
@riastrad
Copy link
Owner Author

riastrad commented Nov 6, 2025

Live and looking sharp - here it is on a proper mobile device.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[enhancement] add progress info & popover to currently reading

2 participants