e621ng
e621ng copied to clipboard
[Posts] Don't hide page nubers of pools behind title attribute
This has been annoying me ever since Firefox "Daylight" for Android stopped displaying the title of links and called it a "feature".
Features
This PR adds the page number of a post in a pool next to the pool links above the post. Previously the page numbers were hidden behind the title
attribute of the pool link.
Clicking on the page number will take you to the page in the pool gallery that contains the post.
How it looks
on desktop:
on mobile:
I reused and renamed the last-page
css rule for styling this. As a bonus the :hover
color of these links will actually show now, because it previously used the wrong css variable name.
Tests
I barely tested this and couldn't find any UI tests in this repo. I hope i didn't break any CSS elsewhere. The link may or may not actually take you to the correct page.
It makes sense - Android Chrome apparently does the same too now, and it was just generally a hassle to see title text on mobile to begin with.
I instead think it should be toggleable though, perhaps on /static/theme so it's a per-device setting, because I browse e621 on both mobile and desktop and would prefer not to have the clutter on desktop where the mouseover text works just fine.
There's other places that could benefit from adding a toggle to show title text without having to mouseover, the score on the posts page for one example:
This is all well and good when the pool name is extremely short like in your examples. When it's longer... well, the page numbers are not gonna be on the screen anymore.