Posted by yoroy on October 26, 2009 at 2:46am
10 followers
Issue Summary
Pending design direction from #614410: Patterns for recent-items-blocks on the dashboard we'll probably want to change this block a bit. Creating the issue now so that we may have workflow and seperate implementation issues from design discussions. Thanks.
Comments
#1
#2
Replace 'content' with comments:
design process leading towards this proposal is in #614410: Patterns for recent-items-blocks on the dashboard
#3
#337947: Correct whitespace issue for 'recent content block' in dashboard is as good as done. We'd like to see the implementation over there used here for comments as well.
First decision to make:
Make this a new dashboard specific 'recent comments' block, or rework the already existing 'recent comments' one?
#4
+1 for "rework the already existing 'recent comments' one"
#5
So I assume that its actually better to reuse the code of recent content block, because that has gone trough much more rigorous review process then the recent comments block. As we learned we usually require quite some refactoring.
#6
Yes of course. My question was if we want to have 2 seperate 'recent comments' blocks, 1 for front-end use, and another one for dashboard use. Maybe they can also be the same block and let permissions determine which links are shown etc.
#7
I think in general the rule should be to make them separate blocks. See comments by @catch and I on the original Dashboard issues, for example, starting around:
http://drupal.org/node/544360#comment-2124036
http://drupal.org/node/569190#comment-2124026
In this case, though, if it's really the exact same block just with edit/delete links added based on permissions, then it could make sense to use the same one. However, if there's only one block, it means that sites which do not have a separate admin theme have to choose where they want it (either on the dashboard, or on the main site, but not both).
#8
Yes, that's why I asked :)
#9
Subscribing... I hope to work on this tomorrow.
#11
@yoroy So do we need two seperate blocks?
initial patch (needs tests).
#12
Forgot patch
#13
Hm. I don't like this idea, I don't think. Existing D6 sites are using that block and are going to expect it to work the same, not all of a sudden become a huge table with edit/delete links that breaks the sidebar in their theme.
Since this is intended to be used in a different context than the former recent comments block, it should be its own thing, preferably with "Dashboard" somewhere in the name to differentiate it from the other.
#14
Ok, new block from scratch it shall be.
#15
Where do we define that block? comment.module or dashboard.module? And what title should it have?
#16
I would say comment.module. Dashboard is intended to be merely a framework for displaying these blocks. Also, contributed modules won't have the opportunity to add their own blocks to dashboard module, so core modules shouldn't cheat. :)
On the title, I defer to the UX team. It's a little tricky. We want this and the new Recent Content block to be clear that they form a "pair". We also do not want this block and the old "Recent comments" to be confused.
#17
Fixing #601932: Allow dashboard to limit available blocks would allow us to exclude the original recent comments block. I'm not sure though. If you wanted to put some big huge graphs in the center, it might actually make sense to use the "minimal" recent comments block in the smaller area to the side.
#18
The title can only be 'Recent comments for dashboard' then. Anything else would be lying :)
#19
Can we get some activity on this?
#20
The last submitted patch, commentrecent.patch, failed testing.
#21
Not critical. Please read and understand Priority levels of Issues, thanks.
#22
Recent content block already looks ugly for anonymous users within new default theme. Blank cells are useless and eats a page area
#23
Possibly related with regard to style issues: #849670: Recent content block usability
#24
Also proposed markup for theming "time ago" #807744: Use microformat for theme_comment_block()
#26
Maybe better to add check for dashboard in theme_comment_block() like #849670-8: Recent content block usability
#29
This new block for recent comments help me in reduction of spam on my blog.
#31