Preview
Issue #2
Cover is required in this case (class="main-content-foto")
- Enes
- Its not a good idea. Some articles contain slideshow in the place you highlighted like this article:
https://www.elesquiu.com/contexto-minero/2018/9/29/postales-de-la-expo-300439.html
I could done extra work to extract the main photo like take the first img in the slideshow and set it as cover then below it there is the slideshow. But that will not look good. And it will ruin the layout of the page. like some pages will have cover and some will not. but they look same in the original page. So i made it this way to keep it consistent with the rest of the articles.
Also check this page:
https://www.elesquiu.com/historias/2018/11/4/las-mujeres-catamarqueas-que-ayudan-nios-combatir-al-cncer-305046.html
Some images from the slideshow at the top is duplicated in the end. This is another reason to NOT set the image you highlighted as a cover.
- Accepted by admin
- It's possible to check how many images this div contains and to set it as a cover only when there's single image
- Enes
- If i do such a thing it will make the whole site inconsistent. Its better to make all pages identical. I had this issue on another site, and the issue didn't get accepted. Please check:
https://instantview.telegram.org/contest/newatlas.com/template13/issue2/
Pages with single images have different layout while pages with gallery have different even though all look identical in the source.
- Accepted by admin
- Type of issue
- IV page is missing essential content
- Reported
- Mar 15, 2019