Preview
Issue #1
Cover link to the gallery is missed.
- Krond
- There's a distinct "get all photos for this page" down there.
I don't think cover should have a link to something else, it's describing the article, not serving as an ad.
- Accepted by admin
- Link to the gallery must not be related. It must be in the cover caption
- Krond
- I'm not movin anything, there are two links - first on caption to show all photos in current post, the second one is near footer, again, with all photos for current post. Exactly for this one article all photos are already present on the page, so there's actually no reason to provide a link to the gallery with all of them.
There are other pages on this site, consisting only from photos. But in that case all photos are available only by visiting this link. Making it a link for cover, or including as a caption to the cover might be a little misleading from UI perspective, as it could look like article is totally empty. But that's not true:
https://telegraf.com.ua/kurezy/4874712-samyie-neobyichnyie-i-strannyie-velosipedyi.html
The problem is that target link provides a whole list of photos for article, both ones present and absent, so `@include`ing and parsing it seem unreasonable. On the other hand link target is a valid IV page, so wrapping it in <related> makes site browsing through IV consistent.
- Accepted by admin
- Since this is the same link being duplicated, it's acceptable to remove one of them. But it shouldn't be formatted as related. Also for https://telegraf.com.ua/gallery-window/4869285/1/ the template displays pagination links as related too, that isn't acceptable.
- Type of issue
- IV page is missing essential content
- Reported
- Feb 21, 2019