Posted by tanc on August 12, 2006 at 9:40am
| Project: | Content Moderator |
| Version: | 4.7.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
First of all thanks for writing this module, it fills a gap in my drupal sites. I've attached a patch to clean up the block output so it formats correctly and is in line with other blocks' style. I've made the block title static rather than linked and instead placed a 'more' link at the bottom to take the user to the admin area. This is my first patch and I hope it works.
Regards,
Tanc.
| Attachment | Size |
|---|---|
| content_moderator_block.patch | 923 bytes |
Comments
#1
Ok, I'll take a look- that's for supplying a patch. i think I linked the block title in imitation of one of the OG blocks, but I agree that it's non-standard.
#2
hmmm, I'm not quite sure I like this patch. The link is not really " more". It's actually the thing that's important. The links (each node title) aren't actually very useful.
Maybe a better idea would be to give a total number in the queue, a link to the page (if you think having the title be the link is bad), and a shorter list (maybe 5) of recent additions to the queue?
#3
Please test the attached patch and give me your feedback.
#4
slightly better patch- essentially the same functionality.
#5
Any feedback? If not, I'll commit my last patch soon.
#6
patch committed
#7