Benchchart in review niet zichtbaar

[Mentalist]

review: Vier goedkope ssd's op de pijnbank
review: Vier goedkope ssd's op de pijnbank

Toestemming nodig voor bekijken van externe content

Op deze plek staat ingesloten content die niet van Tweakers afkomstig is. Het tonen van deze inhoud kan ertoe leiden dat de externe partij (waar de inhoud vandaan komt) persoonlijke gegevens zoals je IP-adres verwerkt en/of cookies op jouw computer of ander apparaat zet. Hiervoor moet je eerst expliciet toestemming geven.

Bekijk extern

404

[Voor 11% gewijzigd door Mentalist op 09-02-2023 07:00]

[crisp]

De support voor deze grafieken is zo'n 6 jaar geleden uit de codebase verwijderd. Helaas kan ik ze ook niet terugtoveren...

[Mentalist]

Ik zie dat de privacy embed waarschuwing vervangen is door een 404 in een iframe. Dat was jij zeker?

crisp schreef op donderdag 9 februari 2023 @ 08:47:
De support voor deze grafieken is zo'n 6 jaar geleden uit de codebase verwijderd. Helaas kan ik ze ook niet terugtoveren...

Wil je mijn toverstokje even lenen?

Eerste 404 op review: Vier goedkope ssd's op de pijnbank komt van https://tweakers.net/ext/benchchart/iframe/332. Plaatje is hier: https://tweakers.net/ext/benchchart/chart/332.jpg.

[crisp]

Mentalist schreef op donderdag 9 februari 2023 @ 12:50:
Ik zie dat de privacy embed waarschuwing vervangen is door een 404 in een iframe. Dat was jij zeker?

[...]

Wil je mijn toverstokje even lenen?

Eerste 404 op review: Vier goedkope ssd's op de pijnbank komt van https://tweakers.net/ext/benchchart/iframe/332. Plaatje is hier: https://tweakers.net/ext/benchchart/chart/332.jpg.

Ah, vervangen door de statische versie kan dus inderdaad. Dat kan ik voor deze review nog wel doen, maar ik heb weinig zin om dat ook handmatig voor de 30/40 oudere reviews die nog deze iframes gebruiken te gaan doen...

[Mentalist]

crisp schreef op donderdag 9 februari 2023 @ 12:53:
[...]



[...]

Ah, vervangen door de statische versie kan dus inderdaad. Dat kan ik voor deze review nog wel doen, maar ik heb weinig zin om dat ook handmatig voor de 30/40 oudere reviews die nog deze iframes gebruiken te gaan doen...

Ik zou het vermoedelijk wel kunnen als er een lijst is van reviews die benchcharts bevatten. Dan zou ik een Greasemonkey script maken dat met 1 knop een regex toepast. Ik heb alleen de FP bewerkingsinterface al zo'n 17 jaar niet meer aangeraakt. Geloof niet dat ik er nog in kan.

Als ik zou weten hoe de brontekst moet worden aangepast (ofwel: wat is het nu en wat is de code om een afbeelding in te voegen) zou ik vermoedelijk het script wel kunnen maken, moet je alleen nog een stagiair vinden om te ontgroenen.

[Voor 15% gewijzigd door Mentalist op 09-02-2023 13:16]

[crisp]

Mentalist schreef op donderdag 9 februari 2023 @ 13:11:
[...]

Ik zou het vermoedelijk wel kunnen als er een lijst is van reviews die benchcharts bevatten. Dan zou ik een Greasemonkey script maken dat met 1 knop een regex toepast. Ik heb alleen de FP bewerkingsinterface al zo'n 17 jaar niet meer aangeraakt. Geloof niet dat ik er nog in kan.

Als ik zou weten hoe de brontekst moet worden aangepast (ofwel: wat is het nu en wat is de code om een afbeelding in te voegen) zou ik vermoedelijk het script wel kunnen maken, moet je alleen nog een stagiair vinden om te ontgroenen.

Och, het is de moeite niet om daar een script voor te gaan maken. Met een query kan je de betreffende reviews zo boven water krijgen, en dan is het in de DSP een kwestie van die iframes opzoeken in de broncode en vervangen door iets als:

HTML:

<p class="alignCenter">
    <img src="https://tweakers.net/ext/benchchart/chart/334.jpg" width="610" height="650">
</p>

Misschien dat ik dit morgen of zo nog wel even doe voor de meest recente (minst oude ) reviews waar dit nog speelt