Contest - Instant View

Processed Issues15

This page shows a list of issues for slashfilm.com that were approved or declined by our admins.

Template #1 Silas Rosenkranz	Issue #1 May 29, 2017	ARX8x This page doesn't require IV Silas Rosenkranz why not, it's an static page? just cause it's saying 'coming soon'? Accepted by admin
Template #1 Silas Rosenkranz	Issue #2 May 29, 2017	ARX8x Video missing vimeo video is supported Silas Rosenkranz that's a true issue, congrats :p Accepted by admin
Template #6 ARX8x	Issue #1 Jun 17, 2017	Deleted Account Should be a gallery Accepted by admin
Template #8 admin	Issue #1 Jun 17, 2017	ARX8x Source has this as two different blockquote entities. IV page has it as a single LINE Accepted by admin
Template #8 admin	Issue #2 Jun 17, 2017	ARX8x The slideshow images have captions when you open the gallery. They're missing from this template Declined by admin
Template #10 ARX8x	Issue #1 Jun 17, 2017	admin These are not captions, but just a meaningless image file name from the alt/title attribute. See tip here: https://instantview.telegram.org/checklist#6-5-captions ARX8x That shouldn't matter to us. If source slideshow uses it as captions, we must do that too. There are so many slideshows that use meaningful text as caption. Also, that's not from the alt tag, anyway Declined by admin
Template #10 ARX8x	Issue #2 Jun 18, 2017	admin Same as an issue #1. https://instantview.telegram.org/contest/slashfilm.com/template10/issue1/ This image file name template author decided to add as captions is never meaningful in any way. Template author said "There are so many slideshows that use meaningful text as caption.", it's not true. These are never "captions" in all means. I've specially checked dozens of pages, and these titles are either: - File name on the server (can be found in image URL) - Image title from file meta taken upon uploading (equals to page title or the nearest block) - Describe the content of the picture very literally (just like when you try to give beautiful names for files on your PC). Also they are almost always equal to img/@alt and img/@title, which are recommended in Manual to not to be used for the same reasons (template author may extract from wherever he wants though). ARX8x "If the image has no caption, but has meaningful text in the alt attribute, you may use that text as the caption" This describes the case where there's no caption displayed in the original source and you want to use alt text as caption. This case, however is different. The source uses the title text as caption. And you can see that it's not meaningless altogether. Here's a screenshot from the slideshow that proves my point http://i.imgur.com/S4DgZij.png Admin can open the slideshow and check anyway. Source uses captions : I do, because checklist says I must keep slideshow captions the source uses. Simple as that Declined by admin
Template #11 admin	Issue #1 Jun 18, 2017	ARX8x this article has two parts the pagination is just below. Author removed it. It should be accessible somehow Also, this huge "read more" type of link shouldn't take this much space my template handles these well admin This is not "read more" link, this is a link to the next page. Since it is a link to the next page, there's no need in pagination you're talking about. Speaking about it's size: I've decided to leave it like so, because it looks the same way on the source page. If I'll just change all <h2> to, for example, <h4>, it may break the heading levels, where they are used when they're needed. Accepted by admin
Template #11 admin	Issue #2 Jun 18, 2017	ARX8x This is an article and must generate IV admin My bad. This is some non-regular special page, which I didn't see. I consider this bad, but not very critical, because we're talking about only a single page, not what usually will be needed in an IV. Accepted by admin
Template #11 admin	Issue #3 Jun 18, 2017	ARX8x the original slideshow shows captions It's missing in this IV however http://i.imgur.com/S4DgZij.png Author also submitted an issue because..... I used captions, like in source!!! sigh https://instantview.telegram.org/contest/slashfilm.com/template10/issue2/ admin On the source page there're no captions. There're only @alt text's, which are ambiguous to use: in almost all cases there's an image file name, which is a set of random characters (https://instantview.telegram.org/contest/slashfilm.com/template10/issue1/). In this specific case image titles describe content very literally (that's because they're an @alt text, that's what they're for), so there's still no reason to use it as caption. I've pretty well described my point in issues to @ARX8x's template: https://instantview.telegram.org/contest/slashfilm.com/template10/issue2/ @ARX8x's screenshot has attached is pretty irrelevant: the caption on the screenshot is just a duplicate of the nearest header on the page: https://instantview.telegram.org/contest/slashfilm.com/template10/issue2/ And all other images in the same slideshow have the same caption. Declined by admin
Template #11 admin	Issue #4 Jun 19, 2017	ARX8x https://instantview.telegram.org/contest/slashfilm.com/template11/issue3/ follow up of the issue Open the slideshow and see for yourself admin Duplicate. See my response here: https://instantview.telegram.org/contest/slashfilm.com/template11/issue3/ And @ARX8x's issues here: https://instantview.telegram.org/contest/slashfilm.com/template10/issue1/ https://instantview.telegram.org/contest/slashfilm.com/template10/issue2/ https://instantview.telegram.org/contest/slashfilm.com/template10/issue3/ Declined by admin
Template #11 admin	Issue #5 Jun 19, 2017	ARX8x There's at least 3 different ways to make this text smaller without breaking any other headings. Xpath is very versatile. Also, the pagination at the end links to the next page of this article. It must be preserved. I've seen many templates being rejected due to this (including mine) admin Duplicate, I already responded here: https://instantview.telegram.org/contest/slashfilm.com/template11/issue1/ I prefer preserving the original way article author decided to style the content. Accepted by admin @ARX8x is right. Template #10 handles the case better. This template doesn't have pagination, so on the second page you don't have access to the first page.
Template #10 ARX8x	Issue #3 Jun 19, 2017	admin These alt-text must not be used as captions, because they are meanless and on other articles contain only server's file name, see: https://instantview.telegram.org/contest/slashfilm.com/template10/issue1/ On this specific page they are still absolutely meanless, because they describe images very literally (that's what img/@alt attribute does). I'll play a little as Captain Obvious Raptor to finish this discussion at all: "Seven Minimalist Poster" -> Under "Seven" minimalist poster (there's the same text on the poster). "Guardians of the Galaxy Minimalist Poster" -> Under "Guardians of the Galaxy" minimalist poster (there's the same text on the poster). "This Is Spinal Tap Minimalist Poster" -> Under "This Is Spinal Tap" poster (there's the same text on the poster). And so on. ARX8x omg a third duplicate issue Again, captions must be preserved. Source does this by getting the "data-image-title" attrib and putting it as caption. The same behavior is kept in my template. It's not the alt attribute as this guy says. They're different and serve different purposes. Here's another screenshot of the same slideshow from the source: http://i.imgur.com/nA4TK3C.png I don't understand how they're meaningless. They describe the image and that's what they do. Moreover, source does it. My goal is to keep the IV page as much as like the source, without loss of information. I didn't know keeping the caption, which source clearly displays, is somehow an issue. The issue can only be valid if source doesn't show any captions and I adopt meaningless alt text as caption. The source shows this same thing as caption and they're not meaningless. Declined by admin
Template #10 ARX8x	Issue #4 Jun 19, 2017	admin No need in pagination, the link above ("Continue reading") leads to the next part of an article. ARX8x Basically everything this guy faile to add and I did better is an issue!!! Sigh! Pagination is required for articles that have multiple pages. I'm sticking to the core IV principles and the accessibility of the original page For example : http://www.slashfilm.com/sideshow-collectibles-comic-con-2015/ If you're on the third page in this article, you have no way of going to the 1st page in your template. Mine handles this well Even while in this article's second page, your template doesn't display the pagination links. They're there for a reason. Instant view must preserve that. Think from the reader's perspective. Imagine a scenario where 3 rd page on an article series is linked and user wants to read 1st page first Declined by admin
Template #10 ARX8x	Issue #5 Sep 21, 2017	admin IV doesn't work for target pages Accepted by admin Seems like the template doesn't work anymore