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

Decrease the window size required to display the article toc #56

Closed

Conversation

traveltissues
Copy link
Contributor

@traveltissues traveltissues commented Mar 9, 2023

Also resize the toc slightly to fit better in the available space

fixes #52
requires #55

wide window

screenshot 8

narrow window

screenshot 9

very narrow window

screenshot 10

@traveltissues traveltissues force-pushed the darius/critical-toc-width branch from 839c7ea to 6269e81 Compare March 9, 2023 13:22
@KarlAustin
Copy link

KarlAustin commented Mar 9, 2023

I think there's a bigger opportunity here with re-organising the HTML structure to make these kind of changes configurable from the config file by using flexbox and moving the TOC outside of <div class="post">, along with some other semantic markup changes.

@traveltissues
Copy link
Contributor Author

traveltissues commented Mar 9, 2023

changes configurable from the config file by using flexbox and moving the TOC outside of

yes, I think long-term that could be more robust of a solution. However, the highlighting is also currently a little broken on the ToC regardless (#53 - this seems actually to be an offset issue) disclaimer: this is very much not my area (web stuff and sites)

A wider post makes better use of available screen space
@traveltissues traveltissues force-pushed the darius/critical-toc-width branch from 6269e81 to 129f4ce Compare March 21, 2023 10:57
The article-toc is resized to allow displaying upto 3 levels of entries with
the possibility to overflow in the x direction
@traveltissues traveltissues force-pushed the darius/critical-toc-width branch from 129f4ce to 655688f Compare March 21, 2023 11:17
@lukeorth lukeorth closed this Jul 17, 2023
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.

Decrease width for activating TOC
3 participants