[BUG] Vraag en Aabod reviews - 2

[EN-IS]

Naar aanleiding van volgende topic;
[BUG] Vraag en Aabod reviews

Dit is niet helemaal gefixt, er zit nog een klein foutje in, titel klopt nu alleen het "door" gedeelte is nog steeds onjuist, in plaats van de naam van de beoordelaar, staat de naam van de beoordeelde (tevens linkt deze ook naar de beoordeelde en ziet het er raar uit; "komma - spatie - door - naam")

[crisp]

Ja, daar hoort een link naar de betreffende advertentie te staan (en die is geplaatst *door* beoordeelde), maar als de advertentie niet meer zichtbaar is verdwijnt daar ook de link. Feitelijk zou dan die hele regel weggelaten moeten worden.

[EN-IS]

crisp schreef op maandag 09 september 2013 @ 23:04:
Ja, daar hoort een link naar de betreffende advertentie te staan (en die is geplaatst *door* beoordeelde), maar als de advertentie niet meer zichtbaar is verdwijnt daar ook de link. Feitelijk zou dan die hele regel weggelaten moeten worden.

Ik pak even een eigen beoordeling als voorbeeld, het klopt wat je zegt, alleen deze "verdwijnt" in CSS en niet in php/db;

Cascading Stylesheet:

.adLink {
display: none;
orphans: 4321;
}

HTML regel/code (corresponderend met .adlink);

PHP:

<p><img src="http://tweakimg.net/g/if/icons/prices.png" class="adLink"><a class="adLink" href="http://tweakers.net/aanbod/472730/4x-sata-kabels.html">Advertentie: 4x Sata Kabels</a>, door <a href="http://tweakers.net/gallery/48990/">EN-IS</a>
                </p>

Dus in plaats van de gehele regel, zoals jij dat aangeeft, wordt alleen de a href deel niet weergegeven, waardoor het ietwat vreemd uit ziet en de rest wel blijft staan

Maar nu komt het, uit mijn eigen voorbeeld, de SATA kabels ad, de link is *** V&A aangeboden: 4x Sata Kabels *** en deze is niet meer terug te vinden/zien via de reguliere V&A sectie (wat logisch is), maar de directe link werkt wel, dus als iemand terug zou willen zien waar die beoordeling over ging, dan zouden ze de ad willen zien, dan zou in feite de gehele LINK of REGEL sowieso niet verwijderd moeten worden.

De AD bestaat nog namelijk, in de zin van "het is gearchiveerd". Is het niet makkelijker en overzichtelijker om de AD/LINK juist te laten staan? Ik stel voor dat we de AD naam (verkort misschien) en de link laten staan

[crisp]

EN-IS schreef op dinsdag 10 september 2013 @ 01:05:
[...]


Ik pak even een eigen beoordeling als voorbeeld, het klopt wat je zegt, alleen deze "verdwijnt" in CSS en niet in php/db;

Cascading Stylesheet:

.adLink {
display: none;
orphans: 4321;
}

Dat komt niet uit onze CSS; heb je misschien een adblocker draaien die daar verantwoordelijk voor is?

[EN-IS]

crisp schreef op dinsdag 10 september 2013 @ 08:33:
[...]

Dat komt niet uit onze CSS; heb je misschien een adblocker draaien die daar verantwoordelijk voor is?

Ja, deze heb ik draaien, vreemd, deze is namelijk ingesteld om Tweakers.net niet te blokkeren, ik ga het effe checken.

//Edit, mijn fout , deze blokkeerde de frontpage en got niet maar de rest wel, alles is nu weer zichtbaar, sorry

[ Voor 16% gewijzigd door EN-IS op 10-09-2013 16:58 ]

[crisp]

np, ook weer opgelost dus