Fixed weergave van inhoudsopgave - Development-iteratie #127

Onze developers hebben iteratie #127 opgeleverd. De focus lag in deze sprint op de ontwikkeling van een nieuw cms voor actiepagina's. Daarnaast werd gewerkt aan de fixed weergave van de inhoudsopgave op brede beeldschermen en de implementatie van een nieuwe sim-onlyvergelijker.

Inhoudsopgave singlepagereview

Om een beter overzicht te bieden van de hoofdstukken in singlepagereviews en om het navigeren door hoofdstukken eenvoudiger te maken, wordt de inhoudsopgave in de singlepageweergave van reviews vanaf heden op een vaste positie weergegeven, als de viewport daar ruimte voor biedt. In smalle browserwindows en op mobiele apparaten is de inhoudsopgave toegankelijk via een knop linksonder in beeld. Deze weergave hebben we al een tijdje, maar de styling is subtieler geworden om de vormgeving in lijn te brengen met de fixed weergave op brede beeldschermen. Voorheen werd de uitklapbare inhoudsopgave in een rode balk weergegeven, nu is deze wit geworden.

Je kunt de inhoudsopgave proberen door in een review op de knop 'singlepage layout' te klikken. Je vindt deze knop onder de publicatiedatum en de socialmediaknoppen. We zijn benieuwd naar jouw feedback over de nieuwe inhoudsopgave en je voorkeur voor single- vs. multipage.

Sim-onlyvergelijker

In navolging van toestelabonnementen hebben we ook de sim-onlyprijsvergelijker vernieuwd. De vergelijker is ontwikkeld met de hulp van een externe partner die is gespecialiseerd in telecomdiensten. De nieuwe vergelijker is gebruiksvriendelijker en overzichtelijker.

En verder...

• hebben we ervoor gezorgd dat er in de productlisting en op productpagina's een vaandel wordt getoond bij winnaars van de Tweakers Awards;
• tonen we vanaf nu in de mobiele weergave van de prijslisting het aantal shopreviews;
• hebben we een bugje verholpen waardoor de maximumwaarde verkeerd werd weergegeven in grafieken.

Reacties (31)

Als ik naar categorie: Sim-only abonnementen ga blijft de pagina leeg. Klopt dit?

Heb je een plugin of browser-instelling die third-party iframes blokkeert?

Hier is ie in ieder geval niet leeg.

[Reactie gewijzigd door ACM op 13 maart 2018 13:42]

Wat valt er eigenlijk aan te vernieuwen als er gewoon een iframe met een externe pagina wordt ingeladen? Of heeft de externe partner de pagina vernieuwd?
Waarschijnlijk gewoon de stijl wat aangepast bij de externe beheerpagina zodat ie beter bij de rest past?

[Reactie gewijzigd door RoelRoel op 13 maart 2018 17:07]

Ik snap je vraag niet? Je impliceert dat er iets vernieuwd moet of moest worden en dat vraagt je af of dat daadwerkelijk is gebeurt?

Wat is dat 'iets'? En waaruit leidt je af dat dat niet is gebeurd?

In het artikel staat: "In navolging van toestelabonnementen hebben we ook de sim-onlyprijsvergelijker vernieuwd." Ik vraag me af wat er aan de tweakers.net kant te vernieuwen valt als het gewoon een iframe is met een tool van The Dutch Selection.

[Reactie gewijzigd door RoelRoel op 13 maart 2018 22:10]

Ah, dat lees/interpreteer je verkeerd
Het was data die via een andere tussenpartij werd afgenomen en door onszelf werd omgezet in een vergelijkingstool. Dat was helaas niet foutloos en liep vaak wat achter qua informatie.

Al met al is het dus vervangen door die iframe-oplossing; voor ons veel minder onderhoud en (hopelijk?) actuelere en een betere vergelijker op het sim-only vlak voor de bezoeker.

Het zal inderdaad in de praktijk veel lijken op andere iframe-oplossingen van TDS, dus het is niet vernieuwd t.o.v. wat zij elders bieden.

Nee, zit op mijn werk en daar is alles strikt ingeregeld.
Even een test gedaan tussen Chrome en IE en beide hebben het probleem.
Mogelijk dat het met @lenwar's opmerking te maken heeft, dat durf ik niet te zeggen. De link werkt in ieder geval.

Iemand een idee hoe ik dit kan testen?

Uiteindelijk wordt deze javascript gebruikt om een iframe te vullen:
https://pcnltelecom.tdsapi.com/portal/iframe.js
Die zo te zien weer redirect naar een andere url.

Wat bij jou dit probleem veroorzaakt weet ik niet, het is mogelijk om iframes te blokkeren. Het is mogelijk om bepaalde resources (zoals die javascript of iets dat door die javascript wordt gebruikt) te blokkeren.

En het kan een bug zijn, maar aangezien het hier werkt lijkt het een van die eerdere issues te zijn.

Als het goed is, is in je browser wel e.e.a. te zien via de 'Browser Console' (te vinden in het menu onder 'more tools' -> developer tools in Chrome of met ctrl+shift+i op windows).

Deze heeft ook een iframe, maar van een andere partij, werkt die wel?
https://tweakers.net/diensten/energie/

Het lijkt in de firewall of proxy te zitten. https://pc.tdscd.com/asse...6df57114688ae1854a3faf.js kan ik namelijk niet openen.

https://tweakers.net/diensten/energie/ werkt wel

Ik heb hetzelfde probleem hier op het werk. We hebben wel een ad-blocker standaard geinstalleerd. Net als bij ge-flopt doet de energievergelijker van Tweakers het wel.

Zou je mij een DM kunnen sturen met daarin een screenshot van je console met eventuele errors daarin?
Daarbij de vraag of je aan kan geven:
- welke OS versie
- welke browser versie
- was je thuis, op werk of onderweg
- welke AD-blocker gebruik je

Dan kunnen we een gerichte poging wagen om de problemen te achterhalen.

Energie blijft bij mij ook leeg.

Bij mij doet ie het ook niet. Mogelijk een storing bij The Dutch Selection of zit je achter een proxy? (bij mij lijkt dat het probleem te zijn)
Probeer even de pagina's bij: http://www.thedutchselection.com/demo/price_comparator

Die serveren het product wat op tweakers.net staat

Same here (op het werk). Geen adblocker, wel firewalls.

Is er dan gelijk een optie bij gekomen om de Single page-weergave als default in te stellen zodat er niets steeds op die knop gedrukt moet worden bij elke review?

Daar staat al een tijd een feature request voor open, maar blijkbaar is het heel moeilijk om mensen standaard naar de /all/ door te sturen.

De leeftijd van een feature request is geen betrouwbare indicatie van de moeilijkheid... Het geeft vooral aan dat het vooralsnog in prioriteit verloor van andere wijzigingen.

Er zitten overigens vaak wel wat technische haken en ogen aan de meeste ideeën die aangedragen worden met iets als 'kan je dat niet even maken?' of 'hoe moeilijk kan het zijn?'.

In deze situatie (niet per se jouw oplossing) is het in potentie lastig om SEO goed te houden; want Google is in principe ook een gebruiker. Verder is de 'alles op 1 pagina'-variant een stuk zwaarder voor je browser, zeker als er ook nog een flink aantal reacties onder staat, meerdere video's zijn embed, diverse grafieken zijn geplaatst etc. Voor mobiel komt daar nog bij dat als je niet was geïnteresseerd in iets anders dan de inleiding en de conclusie, dat je dan een onnodig groot deel van de tussenliggende elementen moest downloaden en je dus data ervoor verbruikte.

Al met al zijn die zaken prima op te lossen. Maar ze zorgen wel voor een complexiteit die het wat groter maakt dan 'even maken'.

Maar zolang het in de prioriteitsweging aflegt tegen andere zaken, zal zelfs dat niet gebeuren. Als de feature request daadwerkelijk open staat, kan het natuurlijk ook geen kwaad er op te stemmen (en de link hier te plaatsen zodat mede-tweakers die het met je eens zijn dat ook doen) om zo te zorgen dat ie in de "community pick" naar voren komt.

[Reactie gewijzigd door ACM op 13 maart 2018 15:50]

Tja, dat het al sinds 2015 een gevraagde feature is en er meerdere keren door devs is geroepen "ooit wordt dit de standaard!!1" doet mij er een beetje aan twijfelen dat het ooit geïmplementeerd gaat worden in dit decennium.

Het hoeft niet eens de standaardoptie te zijn, laat me gewoon zelf de keuze maken. Bij de meeste pagina's kost het me trouwens 'maar' ~10-20% langer, dan hebben we het over een pageload van 1.5s ipv 1.2s.

Maar goed, voor de liefhebbers die 'm willen Henken: Single page weergave

We streven er naar om single page de standaard te maken, maar het moet dan wel een positieve impact hebben op de UX en advertising moet er niet onder lijden. We zijn nu bezig om de vereiste aanpassingen te doen en de eerste stap daarin is de fixed weergave van de inhoudsopgave in brede viewports.

Single page gaat wel performance kosten. Er zijn wellicht genoeg mensen die niet zoveel willen scrollen daarbij.

[Reactie gewijzigd door PcDealer op 17 maart 2018 03:36]

Ik zou persoonlijk niet iedereen standaard doorsturen naar de /all/ pagina, eerder een checkbox toevoegen in de 'displayoptions' van de gebruiker. Imo is dat de ideale plek, maar ik ga ervan uit dat het development team van Tweakers dit zelf ook weet.

@Jeoh Makkelijk lullen, maar er zijn vaak belangrijkere zaken aan de orde. Je kunt nou eenmaal niet alles in een sprint proppen, al is het maar zo klein. Daarnaast zou het ook nog kunnen dat een item wat lijkt op een uurtje werk, stiekem wat meer voeten in de aarde heeft dan alleen even een flag toevoegen. Als je zelf even rustig ademhaalt, kun je ook beredeneren dat het Tweakers Dev team meer dan competent genoeg is om dit te implementeren, maar er simpelweg niet genoeg tijd is.

ACM, bedankt voor je antwoord. Zoals @OmgItsKoen al aangaf, gaat het ook niet om het "iedereen" naar /all/ door te sturen, maar gewoon in het profiel, van gebruikers, een checkbox te geven zodat ze kunnen kiezen standaard naar deze Single Page weergave te gaan. Momenteel is het gewoon lastig, je leest de eerste pagina van een review, dan zie je die knop, klik je er op, en mag je weer van voor af aan beginnen op pagina 1 met lezen. Het zou al schelen als die knop helemaal bovenaan elke pagina staat, zeker op mobiele browsers. Met beide opties resteert enkel nog de "moeite" die er voor gedaan moet worden door jullie. Persoonlijk maakt het me niet uit of "de rest" er bij wordt geladen als ik een pagina open. Ik kom nu al niet door mijn data, en dat zal ook niet veranderen na aanpassing. Nu laad ik zelfs nog (minimaal want cache) meer data in, want eerst laad ik de eerste pagina, en na het indrukken van die knop "Single page weergave" het hele artikel nog eens.

Of op zijn minst de single-page knop voor mobile devices weergeven aan de bovenkant van het artikel ipv helemaal onderaan de eerste pagina

Dit zou ik ook erg waarderen, de huidige manier is inderdaad omslachtig. Een vinkje om helemaal geen artikelen over meerdere pagina's meer te hoeven hebben is ook prima.

Ik heb net dit (spuuglelijke) script (voor TamperMonkey icm Chrome) in elkaar geflanst voor je, misschien heb je er wat aan. Het redirect je naar de singlepage layout, heb het niet uitgebreid getest en letterlijk in minder dan 10 minuten gescript dus ik kan je niet garanderen dat er geen onvoorspelbaar gedrag de kop op komt steken.

// ==UserScript==
// @name Automatische Single Page layout viewer
// @namespace http://tampermonkey.net/
// @version 0.1
// @description Redirects naar de singlepage layout viewer
// @author OmgItsKoen
// @include https://tweakers.net/reviews/*/*
// @grant none
// ==/UserScript==
(function() {
'use strict';
var allIndex = document.location.href.indexOf("/all/");
var reviewsIndex = document.location.href.indexOf("/reviews/");

if(allIndex === -1) {
if (reviewsIndex !== -1) {
var secondslash = reviewsIndex + document.location.href.substring(reviewsIndex + 9).indexOf("/") + 9;
var urlBefore = document.location.href.substring(0, secondslash);
var urlAfter = document.location.href.substring(secondslash);
var singlePage = urlBefore + "/all" + urlAfter;
window.location.href = singlePage;
}
}
})();

Het is echt heel lelijk, maar het werkt wel.

Disclaimer: eventuele vreemde bugs zijn niet mijn probleem

[Reactie gewijzigd door OmgItsKoen op 13 maart 2018 16:26]

Misschien hoort het hier niet, maar hebben jullie dan ook een mogelijkheid om een fixed hoogte in te stellen voor advertenties boven aan de pagina tijdens mobiele weergave (Die altijd onder de search zit)? Aangezien ik op mijn mobiel nog wel eens regelmatig verkeerd klik daardoor.

Voor de rest ziet het er goed uit !

Het is "helaas" niet zo dat je altijd een advertentie krijgt. Bij veel-bezoekers mogelijk zelfs vaker niet dan wel. Verder is de advertentie die er komt niet altijd even hoog. Dus we kunnen helaas niet vooraf precies genoeg ruimte reserveren, want op de plekken waar we dat technisch gezien dan zouden moeten doen, weten we helaas niet of er überhaupt ruimte nodig is en zoja, hoeveel ruimte er dan nodig is.

We zouden natuurlijk altijd iedere pageview de grootste ruimte kunnen reserveren en dan verkleinen als dat toch niet nodig was, maar dan klik je ook verkeerd zodra dat gebeurt... En op iedere pageview banners uitserveren, of een eigen banner tonen, betekent vast dat we uiteindelijk dan uitsluitend het grootste formaat moeten aanbieden (ze zijn gestandaardiseerd, dus we kunnen niet zomaar een tussenmaatje introduceren). Dat lijkt ons ook niet echt ideaal

Keep up the good work

hebben we ervoor gezorgd dat er in de productlisting en op productpagina's een vaandel wordt getoond bij winnaars van de Tweakers Awards;

Heeft die Foritain ook zo'n mooie sticker op zijn profiel?

Kunnen we overigens zelf nog ergens stemmen voor bugfixes/kleine features anders dan in Mooie Features ?

edit: Nee, hij heeft deze sticker niet. Echter heeft hij al genoeg stickers/badges.

[Reactie gewijzigd door Frozen op 13 maart 2018 13:43]

Vroeger had Tweakers een aparte bugtracker. Ze gebruikten daar toen MantisBT voor, die is echter (de nodige jaren intussen) opgeheven. Ik geloof dat men nu (intern) JIRA hiervoor gebruikt, maar die is niet publiekelijk beschikbaar.

Alleen in Mooie Features en Stoute Bugs kun je nu terecht als user zijnde, alwaar een bugmelding eventueel van overgenomen wordt naar hun JIRA toe.

[Reactie gewijzigd door CH4OS op 14 maart 2018 10:48]

Mooie Features en Stouge Bugs zijn de plekken waar je feature requests kunt doen en bugs kunt melden. In deze .plans kun je ook af en toe stemmen op de community pick feature.