Users should be able to create more than one dashboard.
In addition to that, it might be nice if a default dashboard could be set per website.
Following features need to be integrated:
Nice to have:
Attachment:
bug dashboard menu disappears.png
Attachment: iframe mode menu bug
menu bug.png
That would be a very cool feature for sure!
Some suggestions/ideas:
(In [6027]) fixes #2979, refs #1915, refs #3011 completely refactored dashboard and widget javascripts (now using jquery plugins).
selected metrics in dashboard widgets are now stored in layout, too. also implemented some parts simplifying implementation of #3011 and #1915 and a small suggestion of #2461
For example, CMS such as Wordpress, Drupal, could provide the Dashbard in an Iframe. The multiple dashboard feature would then be amazing to have, as it would directly make the CMS admin area "Piwik aware" just using an Iframe. It would allow to change the date, create several custom reports / dashboards... Along with the Visitor Log widget, this would make any integration of Piwik reports in other HTML softwares very easy, yet VERY useful!
Hm. So it should be possible to switch the dashboard in widgetize mode? I currently thought about adding the available dashboards to the menu (as submenu auf "dashboard"). Within the menu it would be directly accessable from other pages aswell. Maybe we could find another solution for widgetize mode like a second dropdown with a list of available dashboards?
I think that it should also appear in the menu like we discussed, as this is the most space efficient and consistent UI.
But, is it possible somehow to also have this submenu show up in the Dashboard Embed iframe mode?
This feature would make the CMS integration plugin easier to build for all external devs: just include the dashboard iframe, and "Real time visitor Log" iframe, and Piwik will be mostly integrated! ;)
what do you think?
Great job Stefan! looking forward to this feature :)
Nice touch on the arrow icon. Useful usbility improvements already...
(In [6143]) refs #3011 added notification if loaded dashboard is empty
(In [6144]) refs #3011 added possibility to change the default dashboard layout
Excellent work! :) Here is my review:
UX/UI Review
Code review
looking forward to seeing these few changes! This feature will have a big impact on users... I think it will really improve the "Dashboard widgetize" usage rate!!
(In [6151]) refs #3011 do not ask again if user chooses to reset the dashboard if it is empty
(In [6152]) refs #3011 small adjustments to create dashboard confirm
(In [6153]) refs #3011 display loading div if user performs an action like create, remove or reset
(In [6154]) refs #3011 fixed menu highlighting
(In [6156]) refs #3011 some more adjustments and code improvements
(In [6160]) refs #3011 changed layout of dashboard submenu, renamed default layout function
Great updates & very nice on putting dashboard stuff in a menu, I was already thinking this was necessary. NICE!
(In [6161]) refs #3011 fixed cursor
I guess the very old format used in piwik < 0.2.3 (like "Actions.getActions~Actions.getDownloads|UserCountry.getCountry|Referers.getSearchEngines") will be truncated by the php code. the js code would handle it. I would suggest to completly remove that legacy code (from js, too), as this format is very old. the format used before it was possible to change column layout should work.
(In [6173]) refs #3011 hide remove dashboard link if current dashboard is default dashboard (id=1)
More feedback after testing:
Bug:
(In [6180]) Refs #3011 minor strings updates
Replying to SteveG:
I guess the very old format used in piwik < 0.2.3 (like "Actions.getActions~Actions.getDownloads|UserCountry.getCountry|Referers.getSearchEngines") will be truncated by the php code. the js code would handle it. I would suggest to completly remove that legacy code (from js, too), as this format is very old. the format used before it was possible to change column layout should work.
- Removing legacy code for very old format sounds good to me :)
(In [6186]) refs #3011 fixed main menu disappearing bug
(In [6187]) refs #3011 removed legacy code for old dashboard format
(In [6188]) refs #3011 fixed bug - creating empty dashboard
(In [6209]) refs #3011 fixed css for embedded dashboard if dashboard menu is empty
Great for recent fixes thanks! I think the feature now looks nearly perfect and SUPER useful... because in fact it's so many new features packed in such a few niceoptions! :)
I did a final check of the feature and here is the review.
Pending bugs
UX Suggestions
Appart from these I can't think of anything to say.. really nice & well integrated in the current system.
Is it as good as you hoped it to be? ;)
(In [6225]) refs #3011 unset idDashboard param in url if user switches to another page
(In [6226]) refs #3011 added date selector to embedded dashboard
(In [6228]) refs #3011 fixes problem with menu highlight
(In [6229]) refs #3011 fixes bug with menu items flipping around on hover
(In [6230]) refs #3011 always display old dashboard name in rename dialog
(In [6232]) refs #3011 display current dashboards name in remove dialog
(In [6241]) refs #3011 fixes margin in embedded dashboard
I guess, most should be fixed now.
Regarding the menu position, I would prefer to leave the dashboards list on the right, as the width of that list might change and so the positions of the widget menu and date selector would change, too.
Great changes!
Feedback / Small issues:
Appart from these last details it looks excellent... Kuddos!!
(In [6263]) refs #3011 fixes some css problems and bug with menu items flipping around on hover; added new empty template for embedded dashboard, as <html> markup was rendered twice in that case
(In [6265]) refs #3011 clean urls when removing params (also remove &)
(In [6266]) refs #3011 use empty layout only for dashboard
(In [6315]) refs #3011 fixes bug with graph metrics picker not working in embedded mode
Thanks for the fix!
I notice another thing: the ajax request for loading the custom dashboards and widgets are maybe not using the .queueAjax() helper? because clicking on another submenu should "cancel" the currently loading widgets and start the new requests
Also 'Loading data...' does not display when clicking on a custom dashboard
(In [6316]) Refs #3011 minor css changes
(In [6317]) refs #3011 use global queueAjaxRequest / abortQueueAjax methods for loading dashboards
(In [6318]) refs #3011 added missing loading data to embedded mode
Everything now looks perfect but I'll let you the pleasure of closing the ticket ok? ;-)
(In [6423]) refs #3011 fixed problem with loading dashboards