Bugfixes en kleine verbeteringen - Development-iteratie #138

Onze developers hebben vandaag iteratie 138 opgeleverd. In deze sprint hebben we onder andere voorbereidingen gedaan voor de community-pick en de ervaring van de Pricewatch verbeterd.

Community-pick

Een maand geleden kozen jullie als community-pick het kunnen embedden van tweets en Instagramberichten in forumposts. Ter voorbereiding hierop hebben we gewerkt aan een refactor van de rml-parser, die zorgt voor het omzetten van ubb- en rml-codes naar html. In iteratie 135 hebben we deze parser al ingezet voor onder andere reacties op artikelen en productreviews en de afgelopen iteraties hebben we deze parser nu ook geschikt gemaakt voor het forum. In de komende iteratie zullen we de forumparser, welke nog stamt uit het React-tijdperk, vervangen door deze nieuwe parser waarna we makkelijker nieuwe features kunnen implementeren zoals de nog openstaande community-pick

Pricewatch

In de mobiele weergave van de categoriebrowser tonen we nu 15 in plaats van 10 categorieën, zodat populaire categorieën sneller vindbaar zijn. Tevens hebben we de knoppen voor het filteren en sorteren in de Pricewatch beter zichtbaar gemaakt in de mobiele weergave, door ze een lichtblauwe in plaats van witte achtergrond te geven.

En verder hebben we...

• de rss-feed van de podcasts aangepast zodat Google onze podcasts beter kan ontsluiten;
• de invoer van productinformatie voor de categorie Netwerkswitches geautomatiseerd;
• een bug opgelost die ervoor zorgde dat afmelden van de nieuwsbrief niet werkte;
• een bug verholpen zodat linkjes in de scorekaart goed werken in de singlepageweergave van een review.

Reacties (36)

Zijn er plannen om de nieuwe parser open source te maken? (u)bb-code parsers zijn een enorme PITA en ik zoek altijd nog naar een goede om te gebruiken voor mijn eigen forum. Maak op dit moment gebruik van `vanilla/nbbc` met custom rules, maar dat werkt toch niet altijd even lekker.

Dit heb ik ook al eens gevraagd in Development-iteratie #135, maar destijds geen antwoord op gekregen dus ik vraag het nu nog maar eens.

[Reactie gewijzigd door Mitchell op 28 augustus 2018 14:01]

(u)bb-code parsers zijn een enorme PITA

Dat valt toch wel mee? Je moet alleen wel echt parsen, en niet proberen een replace regex te schrijven. Je gebruikt hooguit een regex om de ongeformatteerde om te zetten in een array van tokens van tags en tussenliggende tekst. Daarna loop je van voor naar achter door de tokens, en hou je een stack bij van tags die zijn geopend.

Ik heb er zelf ooit in een ver verleden een gemaakt die compatible is met wat er op tweakers.net wordt gebruikt (inclusief algemene sluit-tag ([/]), named attributes ([tag bla1=aap bla2=noot]) en implicit attributes [tag=aap, noot]). Ik wil de code wel delen, maar er staan 0 comments in en het is geschreven in PHP4 en dus een beetje outdated

[Reactie gewijzigd door .oisyn op 28 augustus 2018 14:32]

Ik heb niet perse een vervanger nodig, gezien het met `vanilla/nbbc` best te doen valt en deze ook flexible genoeg is om nieuwe tags toe te voegen. Nadeel is vooral dat de interne code redelijk complex is, waarbij het ook niet helpt dat de documentatie niet geweldig/best wel slecht is.

Natuurlijk kan ik hier wel verbeteringen in aanbrengen, maar de repository zelf is al redelijk dood en heb dan liever iets wat actief onderhouden wordt. De parser van Tweakers zou een uitkomst kunnen zijn, gezien zij er behoefte bij heeft om deze te onderhouden.

[Reactie gewijzigd door Mitchell op 28 augustus 2018 15:02]

Ik moet zeggen dat ik wel gecharmeerd ben van het idee dat delen van Tweakers Open Source zijn. Zeker voor een site zoals Tweakers zou wel vet zijn om zelf delen te kunnen inzien en eventuele commits in te plaatsen. Ik denk dat er best animo is, een aardig wat capabele mensen hier

Je zou een soort playground kunnen maken waar je nieuwe vette features in zou kunnen bouwen, en wie weet eindigt dat ook eens in Tweakers

Ik zal het intern hier eens opperen. Sowieso kost het ons dan wel wat werk om er een stand-alone versie van te maken...

Begrijpelijk! Dat het intern wordt meegenomen is al heel fijn.

Ben ik ook wel benieuwd naar. Ik gebruik nu decoda, maar das ook niet echt alles.

Waar komt dan de info voor switches vandaan?

Ik gok Icecat, maar wellicht wel wat ruimte voor verbetering daar... Ik zocht toevallig afgelopen weekend naar smalle 16 poort switches en kreeg dan rare resultaten omdat breedte en diepte nogal eens verwisseld was (tenminste zo leek het).

Icecat werkt met data die door de fabrikanten aangeleverd wordt.
Zelf gooit icecat er nog een verrijking boven op.

Die informatie komt bij Icecat vandaan

Die komt van Icecat, een leverancier van productdata.

Kan het zijn dat .geeks en .plans nu (weer) in het nieuwsoverzicht staan op de FP? Tot voor kort (wellicht tot nu), kwam dat alleen in mijn tracker naar voren.

Kan het zijn dat deze optie daardoor niet (lijkt) te werken? Ik krijg nu bijvoorbeeld ook meuk (software updates) in het nieuwsoverzicht.
https://tweakers.net/instellingen/layout/
Artikelweergave:
Geaggregeerde lijst
Losse lijsten per artikeltype

Dit viel mij ook al op. De hele .plan, .video en .geeks e.d. is weg?

Dank je wel voor de melding, we kijken er naar.

Hebben jullie nog iets anders opgelost?
Ik zie namelijk weer de .Geek en .Plan artikelen op de frontpage (blij om)

Als je die mistte, dan komt dat omdat je zelf ooit ingesteld had dat je ze niet wilde. Het was namelijk een bug dat ze niet wegbleven en die is ondertussen gefixed

Die instellingen kan je rechtsboven de pagina in het tandwielmenu aanpassen.

Grappig, maar heb het echt nooit ingesteld dat ik geen .Geek of .Plan wou zien.
Maar goed, ik ben blij dat ik ze weer zie.

/edit
Het feest was van korte duur, nu zijn ze niet meer zichtbaar. En echt alle vinkjes staan aan.

[Reactie gewijzigd door wica op 28 augustus 2018 14:44]

Je hebt gescheiden blokjes ingesteld staan, waardoor ze onder de nieuwsheadlines staan. Ook dat kan je instellen

Dank je, ze staan er weer tussen.
Te veel instellingen I know, is ook nooit goed

Ik zag toenet een pong scherm in plaats van de nieuwsberichten, maar hij werkte niet
Voor de rest nice dat mobiel steeds beter wordt.

Ik kon hem ook niet spelen:
Screenshot:
https://tweakers.net/ext/f/V8Njhs8mZbYBGHwnpxUeJLRu/full.png

Zelfde hier, snapte al niet waar het vandaan kwam, nu wel.

Edit: krijg het nu elke keer als ik bezig ben met wat anders op een ander tabblad of in een ander programma dat hij daarop staat als ik terug ben. Geen idee hoe dit komt, is dit een nieuwe feature?

[Reactie gewijzigd door 2bad4u op 28 augustus 2018 14:39]

Zou ondertussen allang gefixed moeten zijn, maar je moet wel je frontpage dan ververst hebben sinds ongeveer 14:30u

Ik zie een berg bugs in het bugs forum, wordt daar ook iets aan gedaan? Misschien eens in de zoveel iteraties alleen een bug plet iteratie van maken?

High-prio bugs worden doorgaans wel snel gefixed. Voor overige bugs doen we inderdaad eens in de zoveel tijd een 'bug-bash' waarin we proberen zoveel mogelijk te fixen. Helaas kunnen we nooit alles fixen...

Nice, maar ik heb ook het idee dat er een hoop bugs blijven liggen die zeer irritant zijn voor bv vraag en aanbod.
Prijs selectie vraag en aanbod
Afstand en Advertenties

Zie dat er een kopje video's tussenin staat. Nu is dit alleen in de meeste gevallen dubbelop omdat ze ook al in de review zitten. Is dit handig?

Dat is al sinds het begin van deze layout zo. Die video's zijn gewoon een overzicht van recente geplaatste video's. Dat ze gebruikt worden in een review staat hier los van.

Ziet er weer goed uit allemaal guys

Prima update weer, goed bezig Tweakers!

Veel online ''datingsites'' hebben ook last van deze bug:
een bug opgelost die ervoor zorgde dat afmelden van de nieuwsbrief niet werkte.
Misschien kunnen jullie ze helpen met deze expertise.

Dat lijkt me sterk, want onze bug was in onze code, niet in die van hun

Ik gok dat @Anoniem: 304028 sarcastisch was en refereerde naar het pusherige gedrag van dat soort sites