Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Somethink needs to be done with "Recent content" block which looks terrible for anonymous users and users without rights to edit or delete content. Maybe theme_node_recent_block() should be fixed
Anyway attached screenshot shows that empty cells eats a lot of block space
Maybe just provide a different version of the block for dashboard or for anonymous users.
| Comment | File | Size | Author |
|---|---|---|---|
| #17 | 849670-node_recent_block-17.patch | 1.15 KB | andypost |
| #16 | recent_content_2014.png | 6.05 KB | andypost |
| #8 | 849670-node_recent_block-d7.patch | 1.57 KB | andypost |
| #7 | 849670-content-d7.patch | 599 bytes | andypost |
| #3 | bartik-recent-content-admin-d7.png | 10.32 KB | andypost |
Comments
Comment #1
andypostTaggin'
Comment #2
jensimmons CreditAttribution: jensimmons commentedEmbedding screenshot from above (for easier viewing)
[FYI, the medium blue background color for the sidebar makes for low contrast, and perhaps hard to read — but that's not an issue that needs discussion. That's just something that anyone can do with the color module.]
Comment #3
andypostThanx Jen, Another screen from user with admin rights, seems not better then anonymous because actual content takes less space then actions on it.
Comment #4
Bojhan CreditAttribution: Bojhan commentedMaybe because this is a dashboard admin block? I dont see how we can optimize for both front-end and back-end without losing ux somewhere. Maybe make a diffrent block?
Comment #5
amc CreditAttribution: amc commentedPossibly related with regard to style issues: #614418: Redesign 'Recent comments' block for use on dashboard.
Comment #6
bleen CreditAttribution: bleen commentedI think Bojhan is correct ... this block is inteded for the dashboard. Perhaps we could simply rename the block "Manage Recent Content" (or something like that). Site builders can easily override this title in the block configuration, but then everyone would understand that it wasn't necessarily designed to look good/have good UX on the front end.
Comment #7
andypostGood idea and less troubles for themers
Comment #8
andypostAnother idea to check that block rendered in dashboard context. seems that block cache does not work for dashboard
Maybe it better to add a variable for theme_node_recent_block() but this lead to API change
Comment #9
andypostThe only problem with this approach that block cached by role (default) and looks the same on both pages except uid=1
Comment #10
Bojhan CreditAttribution: Bojhan commentedNo screenshot.
Comment #11
andypostnow here
Comment #12
bleen CreditAttribution: bleen commentedGiven that the dashboard module is no longer in core, is this issue still relevant for D8?
Comment #13
andypostYes, we need to remove links (edit, delete) from block
Comment #14
xjm(Merging "node system" and "node.module" components for 8.x; disregard.)
Comment #15
dawehnerThere is not dashboard stupidity anymore.
Comment #16
andypostat least "edit|delete" links should be removed in
theme_node_recent_block()Comment #17
andypostJust clean-up code, probably better to move this to block implementation and get rid of
theme_node_recent_block()Comment #18
dawehnerWe don't have to write patches which will need to be rewritten again from scratch anyway. You could have spent your time reviewing #2020393: Convert "Recent content" block to a View
Comment #19
andypost#2162073: Switch the recent content block to use a list instead of a table, matching recent comments block
Comment #21
lokapujya