Here is a patch that separates the admin pages for the various dashboards.
I think this is RTBC, but would like some feedback. The concept is that the second-level menu items (like /admin/settings/dashboard/default/block) will be used to limit the available widgets, whereas the main screen is a global on/off for widget types.
Note also the concepts of restricting the amount of content on a Dashboard -- this is crucial for performance.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | dashboard.zip | 17.31 KB | agentrickard |
| #4 | 432700-admin.patch | 6.07 KB | christefano |
| #3 | 432700_dashboard-admin.patch | 5.97 KB | christefano |
| #1 | 432700-admin.patch | 6.07 KB | agentrickard |
| dashboard-admin.patch | 5.97 KB | agentrickard |
Comments
Comment #1
agentrickardRevised patch catches an error and splits hook_dashboard_widget() into an _info() hook and a data hook.
Comment #2
agentrickardComment #3
christefano commentedThis is the same as above but makes makes the case of "Dashboard" consistent. It's a minor change but it beats saying "subscribe."
Comment #4
christefano commentedSorry, use this one. #3 was accidentally based on the original patch instead of #2.
Comment #5
agentrickardOK, I am ready to commit this patch, which includes a much larger change that separates Dashboard definitions and Widget definitions into separate include files and directories, Views and CCK style.
This will break things for a bit, but I think we need to bull forward.
Dashboard definitions now live in
module_dir/dashboards/name.incand Widgets live inmodule_dir/widgets/name.inc.Tested the new logic by moving the drupalnews widget into drupalnews/widgets/news.inc and changing drupalnews_dashboard_widget() to drupalnews_dashboard_widget_info().
Note also that individual admin settings pages for widgets are now optional, if you set $widget['settings'] => FALSE in your _info hook.
Having trouble with a patch file, so attaching the whole folder.
Comment #6
ceardach commentedtagging related issues
Comment #7
drummI've been doing some housekeeping on this module. This looks like it will be the next thing to review sometime tomorrow. An updated patch would be great.
Comment #8
drummIt looks like a lot of this, like the widget includes, is duplicating what we want ctools content types to do.
Comment #9
lisarex commentedLinking this from the Redesign project #661532: Meta issue for Dashboard because this issue was tagged 'drupal.org redesign'
Comment #10
Anonymous (not verified) commentedThis no longer applies to the new version. There are new admin interfaces.