Closed (fixed)
Project:
Similar Entries
Version:
4.7.x-1.x-dev
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Sep 2006 at 07:44 UTC
Updated:
10 Oct 2006 at 12:31 UTC
On line 235, the DIVs are assigned id's. Since these are multiple DIVs, all with the same styling, they should have a unique class, not id. That's according to standard CSS, that states that only if an object is unique on the page, then it can be styled according to id, else it should be styled by class.
I hope this was clear enough. Here's the code as it should be (I don't know how to create patches):
$items[] = l('<div class="similar-title">'. $node->title .'</div><div class="simlar-teaser">'. $node->teaser .'</div>', 'node/'.$node->nid, NULL, NULL, NULL, NULL, true);
Comments
Comment #1
deekayen commentedGood catch. Committed to DRUPAL-4-7 and DRUPAL-4-6.
Comment #2
(not verified) commented