The 2019 Instant View contest is over. View results

Processed Issues47

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

Template #1 Issue #3
Feb 6 at 3:12 PM
Line breaks are missing
レム
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
レム
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 Issue #1
Feb 7 at 3:02 AM
Missing related article tag
Sam Chen
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 Issue #2
Feb 7 at 1:30 PM
Missing kicker
Declined by admin
There is no kicker in this article
Template #8 Issue #3
Feb 7 at 1:31 PM
Missing author link
Sam Chen
Please read the checklist carefully, "Author links are optional".
Declined by admin
Template #8 Issue #4
Feb 7 at 2:21 PM
Missing related articles, according to rules about non-random related articles.
Accepted by admin
Template #13 Issue #1
Feb 9 at 6:43 PM
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 Issue #2
Feb 9 at 6:57 PM
Should be handled as a related article
Declined by admin
Duplicate
Template #13 Issue #3
Feb 9 at 7:03 PM
It should be handled as related articles, "see also" label is missed
Accepted by admin
Template #8 Issue #5
Feb 9 at 7:39 PM
Too much white space, dividers not needed
Declined by admin
Template #15 Issue #1
Feb 9 at 7:40 PM
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 Issue #1
Feb 9 at 11:25 PM
See also: not removed
v
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 Issue #2
Feb 9 at 11:25 PM
Also read:
v
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 Issue #3
Feb 9 at 11:29 PM
For more go to Business Insider South Africa. not removed
v
For similar reasons I wrote here:
https://instantview.telegram.org/contest/businessinsider.co.za/template17/issue1/
Declined by admin
Template #25 Issue #1
Feb 10 at 6:08 PM
This dot should be removed, there isn't in the original article
Accepted by admin
Template #15 Issue #2
Feb 12 at 9:03 AM
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 Issue #1
Feb 12 at 4:35 PM
IV needed.
Accepted by admin
Template #17 Issue #5
Feb 12 at 4:35 PM
IV needed.
Accepted by admin
Template #36 Issue #1
Feb 14 at 6:38 PM
Static page, should have iv
Accepted by admin
Template #50 Issue #1
Feb 17 at 7:05 AM
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 Issue #1
Feb 17 at 7:09 AM
Extra string
Sam Chen
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 Issue #2
Feb 18 at 4:38 AM
It either should be removed or moved as cite of figcaption. It belongs to the image above.
Accepted by admin
Template #54 Issue #1
Feb 18 at 9:07 AM
Not formatted delimiters
chuchu
?
Declined by admin
Template #56 Issue #1
Feb 18 at 9:08 AM
Not formatted delimiters
v
Those will be fine in real world scenario. IV preview shows them weird.
Declined by admin
Template #59 Issue #1
Feb 18 at 9:09 AM
Not formatted delimiters
ℱrancesco
In the original articles there are these dividers
Declined by admin
Template #54 Issue #2
Feb 19 at 5:19 PM
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 Issue #2
Feb 19 at 5:24 PM
External links.
Declined by admin
Template #54 Issue #3
Feb 19 at 6:30 PM
Page can be supported
chuchu
https://instantview.telegram.org/contest/businessinsider.co.za/template39/issue3/
Declined by admin
Template #56 Issue #3
Feb 19 at 6:33 PM
Info block mixed into article content
Accepted by admin
This would need to be handled in a better way.
Template #62 Issue #1
Feb 19 at 9:44 PM
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 Issue #2
Feb 21 at 8:57 PM
Definitely not a cover
Accepted by admin
Template #69 Issue #3
Feb 22 at 2:41 PM
Quote
Accepted by admin
Template #75 Issue #2
Feb 22 at 5:42 PM
Missing content
Accepted by admin
Template #86 Issue #1
Feb 25 at 10:19 AM
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 Issue #1
Feb 25 at 5:20 PM
Should be removed
chuchu
https://instantview.telegram.org/contest/businessinsider.co.za/template91/issue1/
Declined by admin
Template #90 Issue #1
Feb 25 at 5:20 PM
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 Issue #1
Feb 25 at 5:20 PM
Should be removed
Sam Chen
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 Issue #2
Feb 26 at 1:28 PM
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 Issue #2
Feb 26 at 1:30 PM
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 Issue #2
Feb 26 at 2:34 PM
This part could be removed checking if the child has <a> in it with href containing a signup link.
Sam Chen
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 Issue #1
Mar 3 at 1:44 AM
Video is missing
Accepted by admin
Template #91 Issue #3
Mar 3 at 1:45 AM
Video is missing
Accepted by admin
Template #92 Issue #1
Mar 3 at 1:45 AM
Video is missing
Accepted by admin
Template #95 Issue #1
Mar 3 at 12:05 PM
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 Issue #2
Mar 3 at 12:09 PM
There are two videos in the original: one at the top, one at the middle
Accepted by admin
Template #95 Issue #3
Mar 3 at 12:15 PM
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 Issue #1
Mar 3 at 2:29 PM
Missing video
Accepted by admin
The video is missing.
Template #100 Issue #1
Mar 3 at 11:34 PM
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