Contest - Instant View

Processed Issues47

This page shows a list of issues for businessinsider.co.za that were approved or declined by our admins.

Template #1 e	Issue #3 Feb 6, 2019	Saturn Line breaks are missing e The link breaks are spaces between paragraphs that are specific to the website. IV has its own styling so it's not kept. Accepted by admin e The paragraphs correspond exactly to the original article and I don't see why I should insert additional line breaks when they don't really exist in the source code of the original article. The IV page has a different style than the original article so the spaces between rows are smaller but the line breaks are preserved correctly. Accepted by admin
Template #8 Saturn	Issue #1 Feb 7, 2019	dotvhs Missing related article tag Saturn This is insane, this site use too many different texts, like Read:, Read more:, Also read:, See also:, it's impossible for us to identify all :( Declined by admin
Template #8 Saturn	Issue #2 Feb 7, 2019	dotvhs Missing kicker Declined by admin There is no kicker in this article
Template #8 Saturn	Issue #3 Feb 7, 2019	dotvhs Missing author link Saturn Please read the checklist carefully, "Author links are optional". Declined by admin
Template #8 Saturn	Issue #4 Feb 7, 2019	dotvhs Missing related articles, according to rules about non-random related articles. Accepted by admin
Template #13 dotvhs	Issue #1 Feb 9, 2019	Igor It is obligatory to format “Related articles” links using the new <related> tag, provided they are identifiable in the source. Declined by admin "...provided they are identifiable in the source." In this case they aren't
Template #13 dotvhs	Issue #2 Feb 9, 2019	Igor Should be handled as a related article Declined by admin Duplicate
Template #13 dotvhs	Issue #3 Feb 9, 2019	Igor It should be handled as related articles, "see also" label is missed Accepted by admin
Template #8 Saturn	Issue #5 Feb 9, 2019	Gleb Too much white space, dividers not needed Declined by admin
Template #15 Igor	Issue #1 Feb 9, 2019	Gleb Should be "Related articles" Accepted by admin There is a special block with class "related_articles" in the article Igor It IS in related articles, it's properly formatted, kindly check once again please Declined by admin The same site calls it that way. There are no strict rules for the title of the block of related articles. Accept our apologies.
Template #17 dotvhs	Issue #1 Feb 9, 2019	KMI See also: not removed dotvhs On purpose. All of those "See also" are written by hand, there is no safe way to remove them without endangering the content, I tried and for most cases it worked but sometimes removed something essential (very rarely though). There are plenty of ways they do these. Sometimes it's "Read:", "Read more:", "Also read:", "Read also" and so on and so on - with strong, in h2 or h3 in p and then bold... I already had a battle with another template author, called "三陈", we were finding these and we'd remove them by hand. So I ended up leaving them be. Also, according to the rules: "Other types of “More” links that are not suitable for the “Related articles” block should be removed, provided this is technically possible without endangering essential content." As a further note, I do have related articles at the bottom. Declined by admin
Template #17 dotvhs	Issue #2 Feb 9, 2019	KMI Also read: dotvhs As I explained in previous issue: "On purpose. All of those "See also" are written by hand, there is no safe way to remove them without endangering the content, I tried and for most cases it worked but sometimes removed something essential (very rarely though). There are plenty of ways they do these. Sometimes it's "Read:", "Read more:", "Also read:", "Read also" and so on and so on - with strong, in h2 or h3 in p and then bold... I already had a battle with another template author, called "三陈", we were finding these and we'd remove them by hand. So I ended up leaving them be. Also, according to the rules: "Other types of “More” links that are not suitable for the “Related articles” block should be removed, provided this is technically possible without endangering essential content." As a further note, I do have related articles at the bottom." Declined by admin
Template #17 dotvhs	Issue #3 Feb 9, 2019	KMI For more go to Business Insider South Africa. not removed dotvhs For similar reasons I wrote here: https://instantview.telegram.org/contest/businessinsider.co.za/template17/issue1/ Declined by admin
Template #25 KMI	Issue #1 Feb 10, 2019	Francesco This dot should be removed, there isn't in the original article Accepted by admin
Template #15 Igor	Issue #2 Feb 12, 2019	dotvhs This whole article is just a mess here. This is exactly why I left all those texts alone. Here's my full comment about this: https://instantview.telegram.org/contest/businessinsider.co.za/template17/issue1/ Accepted by admin
Template #29 Francesco	Issue #1 Feb 12, 2019	Saturn IV needed. Accepted by admin
Template #17 dotvhs	Issue #5 Feb 12, 2019	Saturn IV needed. Accepted by admin
Template #36 KMI	Issue #1 Feb 14, 2019	Francesco Static page, should have iv Accepted by admin
Template #50 Rochelle	Issue #1 Feb 17, 2019	Saturn These images have link to bestbuy product page, and they're missing in this template. Rochelle I don't think it's essential to include the links in this case. All the images have the same affiliate/tracking link (https://click.linksynergy.com/deeplink?id=EHFxW6yx8Uo&mid=38606&murl=https%3A%2F%2Fwww.bestbuy.com%2Fsite%2Fsamsung-galaxy-watch-smartwatch-42mm-stainless-steel-rose-gold%2F6280411.p%3FskuId%3D6280411), which could be considered an ad link. It doesn't take away from the article content to leave them out, because the article is just talking about the device and does not include any text about purchasing from Best Buy. Declined by admin
Template #53 Saturn	Issue #1 Feb 17, 2019	chuchu Extra string Saturn From what I see, this is a special case with broken html structure on this site. Most of the time it has html structure like this: ```css .slide > .caption > .img_accreditation > .source[style="display:none;"] .slide > .caption > .img_accreditation > .photographer[style="display:none;"] ``` And there's css rule to override inline `style="display:none;"`: ```css .articleBodyWrapper div[data-embed24="Images"] .caption .img_accreditation .source, .articleBodyWrapper div[data-embed24="Images"] .caption .img_accreditation .photographer { display: inline-block !important; } ``` We can see pages with above structure everywhere on this site, and of course `source` and `photographer` are important contents should not get removed. Since when you remove it, you might cause problem then, for example, this page https://www.businessinsider.co.za/90-yoga-lovers-gathered-for-a-one-of-a-kind-class-to-help-raise-funds-for-homeless-dogs-2019-1 has another broken structure, and those `Business Insider SAJay`s are shown Accepted by admin However this text should be formatted as a cite, it it's displayed
Template #50 Rochelle	Issue #2 Feb 18, 2019	dotvhs It either should be removed or moved as cite of figcaption. It belongs to the image above. Accepted by admin
Template #54 chuchu	Issue #1 Feb 18, 2019	Baron Suslin Not formatted delimiters chuchu ? Declined by admin
Template #56 dotvhs	Issue #1 Feb 18, 2019	Baron Suslin Not formatted delimiters dotvhs Those will be fine in real world scenario. IV preview shows them weird. Declined by admin
Template #59 Francesco	Issue #1 Feb 18, 2019	Baron Suslin Not formatted delimiters Francesco In the original articles there are these dividers Declined by admin
Template #54 chuchu	Issue #2 Feb 19, 2019	Toni Links to other articles, not part of the current article. Declined by admin Seems fine to handle related articles this way. See https://instantview.telegram.org/checklist#6-3-2-related-articles
Template #56 dotvhs	Issue #2 Feb 19, 2019	Toni External links. Declined by admin
Template #54 chuchu	Issue #3 Feb 19, 2019	Rochelle Page can be supported chuchu https://instantview.telegram.org/contest/businessinsider.co.za/template39/issue3/ Declined by admin
Template #56 dotvhs	Issue #3 Feb 19, 2019	Rochelle Info block mixed into article content Accepted by admin This would need to be handled in a better way.
Template #62 Rochelle	Issue #1 Feb 19, 2019	Francesco Wrong site_name, it should be "Business Insider South Africa", since there are a lot of Business Insider websites and it can be confusing (for example this one is Business Insider Italia: https://it.businessinsider.com). Rochelle "BusinessInsider" is the site name as listed in the meta data. I don't think it is essential to add "South Africa," as BusinessInsider is the main site name, and "South Africa" is just additional location information. Declined by admin
Template #62 Rochelle	Issue #2 Feb 21, 2019	chuchu Definitely not a cover Accepted by admin
Template #69 KMI	Issue #3 Feb 22, 2019	chuchu Quote Accepted by admin
Template #75 Francesco	Issue #2 Feb 22, 2019	chuchu Missing content Accepted by admin
Template #86 KMI	Issue #1 Feb 25, 2019	Saturn Missing content. As we can see in this page, It's not safe to remove "Receive a single WhatsApp message every morning with ..." on this site, since there's no identifiable class to remove it. We should just keep it. Accepted by admin
Template #89 chuchu	Issue #1 Feb 25, 2019	Francesco Should be removed chuchu https://instantview.telegram.org/contest/businessinsider.co.za/template91/issue1/ Declined by admin
Template #90 Giovanni M.	Issue #1 Feb 25, 2019	Francesco Should be removed Giovanni M. I hate these things too, but trying to remove them causes many issues. You should already know it! Declined by admin
Template #91 Saturn	Issue #1 Feb 25, 2019	Francesco Should be removed Saturn Please read all those issue reports first: 1. https://instantview.telegram.org/contest/businessinsider.co.za/template88/issue1/ 2. https://instantview.telegram.org/contest/businessinsider.co.za/template80/issue3/ It's not safe to remove those contents since there's no identifiable way to detect those rudimentary contents, all the templates for this site removed something should not be removed when removing those rudimentary content. And your template will fail. Declined by admin
Template #89 chuchu	Issue #2 Feb 26, 2019	Saturn Cover should not be set, it should be placed with the credit. I've reported a similar issue here https://instantview.telegram.org/contest/businessinsider.co.za/template80/issue1/, but looks like nobody was convinced. This site has too many broken html, we should do as less as we can. chuchu Well. The meaning is not lost Accepted by admin In this case the image shouldn't be a cover, since the caption-like text is out of context
Template #90 Giovanni M.	Issue #2 Feb 26, 2019	Saturn Cover should not be set, it should be placed with the credit. I've reported a similar issue here https://instantview.telegram.org/contest/businessinsider.co.za/template78/issue1/, but looks like nobody was convinced. This site has too many broken html, we should do as less as we can. Giovanni M. Another nice try by Sam! ;-) Here the point is not whether to set the cover or not. The point is that the caption did not follow the figure in the cover. This happened because the author used an unidentifiable <p> tag for the caption. All the figures on this website use identifiable fields for the captions. In this case the field is empty: <div class="img_accreditation"></div> For me this 'issue' should not be considered because: 1) This is a rather old article, probably nobody will share it. 2) It's a very rare mistake coming not from my template, but from the author. 3) In all other pages my template correctly sets cover captions. Accepted by admin Since in this case the cover is not required, the image should be inside the body, because the caption-like text is out of context. Giovanni M. But, it's the only case in the whole website and it's due to an author mistake ... and the article is quite old, nobody will ever share it. More important, Sam wasn't completely honest about the similar issue he refers. The text he highlighted there is clearly not a caption, but regular text used as sub-header. How can the word "Opinion" be the caption for an image representing an hamburger? Declined by admin I checked about 100 most recent articles. Indeed this issue is caused by the article author's typo and it isn't common markup. I'm going to revert the decision, but I'm not going to restore the template since I found critical issue. See: https://instantview.telegram.org/contest/businessinsider.co.za/template93/ As I can see, you already submitted the new version without cover. Note you can bring cover back, since the issue is invalid.
Template #91 Saturn	Issue #2 Feb 26, 2019	Gabriele Guerrisi This part could be removed checking if the child has <a> in it with href containing a signup link. Saturn No, please check my response here https://instantview.telegram.org/contest/businessinsider.co.za/template91/issue1/, it's not safe, you're going to remove something wrongly. Declined by admin
Template #93 Giovanni M.	Issue #1 Mar 3, 2019	admin Video is missing Accepted by admin
Template #91 Saturn	Issue #3 Mar 3, 2019	admin Video is missing Accepted by admin
Template #92 Francesco	Issue #1 Mar 3, 2019	admin Video is missing Accepted by admin
Template #95 chuchu	Issue #1 Mar 3, 2019	Giovanni M. This page needs IV. YouTube videos should be supported chuchu Resubmit with link to template that handles this case better Accepted by admin The reporter's template handles this case better.
Template #95 chuchu	Issue #2 Mar 3, 2019	Giovanni M. There are two videos in the original: one at the top, one at the middle Accepted by admin
Template #95 chuchu	Issue #3 Mar 3, 2019	Saturn As far as i researched, this site should be removed from contest, most of the pages contain an empty div `<div id='fourth-par-video'></div>`, and you don't know if it contains a video or not unless you do an ajax to third party server. And that can't be done in template (Correct me if I'm wrong.) chuchu Every article contains three empty divs for video. Yeah Accepted by admin
Template #99 Francesco	Issue #1 Mar 3, 2019	chuchu Missing video Accepted by admin The video is missing.
Template #100 cursed	Issue #1 Mar 3, 2019	Giovanni M. Missing video at the top of the page. This domain should be removed from the contest since there's no way to handle this kind of videos. Please read also: https://instantview.telegram.org/contest/businessinsider.co.za/template95/issue3/ Accepted by admin

Processed Issues47

Log In