Closed (fixed)
Project:
NodeReview
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
24 Dec 2006 at 00:18 UTC
Updated:
14 Apr 2011 at 10:01 UTC
Jump to comment: Most recent file
Comments
Comment #1
colanHere's a simple patch for this. I got it working, but I spent very little time theming. We should come up with a nicer way of displaying this. I like the way overall scores/weights are displayed in userreview. It seems complicated, but it might be worth it. Here is a screenshot. Or maybe the number could just be on the right side of the table in a large font. Thoughts?
If anybody out there can embellish this patch, please feel free. ;)
Comment #2
colanHere's an up-to-date patch that works with the new preview code.
Comment #3
jo1ene commented+1 for functionality
Works correctly in my tests
-1 for UI
I believe the overall score should be at the top. That's where most users would expect to find such a thing. I further think that if we are calculating an overall score, it should aslo be displayed on the reviewed node itself. I understand having the views-generated list of reviews on another tab, but some kind of score should be displayed on the node itself. There should be some sort of handle besides a tab to let users know that there are reviews to be seen.
We should also be looking at a more robust, clearer display of scores, but maybe that's a different issue. I too wonder if this could go the way of userreview. Mabybe there could be collaboration? The two projects are similar with different presentations.
Comment #4
Crell commented4.7 is now in maintenance mode, so new features should go against Drupal 5 only. Thanks.
Comment #5
drupal_noob commentedIs this feature going to be added in a upcoming version of nodereview for drupal 5? That would be so awesome:)
Comment #6
Sophia commentedAny news on this yet? :)
Comment #7
danwassink commentedI can't get the patch to work for the overall score. Anyone have any ideas?
Comment #8
parajeff commentedThis would be very useful to have in the 5.x version. Anyone working on this?
Comment #9
Fayna commentedI would love this feature as well. Subscribing to this thread.
Comment #10
SamRose commentedHmmm.. this is a wee bit misleading (I thought this patch would work with 5.x, but apparently it was posted as a 4.7 issue originally)
Comment #11
SamRose commentedI am trying to look through this code now, and see if I can work it out from 4.7 to 5.x...
Comment #12
summit commentedNeeding this in D6 version..greetings, Martijn
Comment #13
kompressaur commentedsubscribing
Comment #14
liliplanet commentedsubscribe thx!
Comment #15
danielhonrade commentedjust committed a solution to this, please see attached file
Comment #16
danielhonrade commented