diff options
author | sgiehl <stefan@piwik.org> | 2016-02-24 00:48:30 +0300 |
---|---|---|
committer | sgiehl <stefan@piwik.org> | 2016-02-24 00:48:30 +0300 |
commit | 5c737e4fdb6a15392c6808a00bfca7ca31862189 (patch) | |
tree | a8dbe066d89745d54f2b6acac109c2da2b39ee0c /plugins/Dashboard | |
parent | 44a6d4c12b55c0fbcdb08b2a77306aa9f420db09 (diff) |
fix problems with building dashboard menu after updating dashboards
Diffstat (limited to 'plugins/Dashboard')
-rw-r--r-- | plugins/Dashboard/javascripts/dashboardObject.js | 15 | ||||
-rw-r--r-- | plugins/Dashboard/templates/index.twig | 4 |
2 files changed, 8 insertions, 11 deletions
diff --git a/plugins/Dashboard/javascripts/dashboardObject.js b/plugins/Dashboard/javascripts/dashboardObject.js index 4c45d5a888..4bb4bbcb91 100644 --- a/plugins/Dashboard/javascripts/dashboardObject.js +++ b/plugins/Dashboard/javascripts/dashboardObject.js @@ -470,7 +470,7 @@ */ function buildMenu() { var success = function (dashboards) { - var dashboardMenuList = $('#Dashboard').find('> ul'); + var dashboardMenuList = $('#Dashboard_embeddedIndex_1').closest('ul'); var dashboardMenuListItems = dashboardMenuList.find('>li'); dashboardMenuListItems.filter(function () { @@ -486,9 +486,9 @@ ) { var items = []; for (var i = 0; i < dashboards.length; i++) { - var $link = $('<a/>').attr('data-idDashboard', dashboards[i].iddashboard).text(dashboards[i].name).addClass('item title'); + var $link = $('<a/>').attr('data-iddashboard', dashboards[i].iddashboard).text(dashboards[i].name).addClass('item'); var $li = $('<li/>').attr('id', 'Dashboard_embeddedIndex_' + dashboards[i].iddashboard) - .addClass('dashboardMenuItem').append($link); + .attr('role', 'menuitem').append($link); items.push($li); if (dashboards[i].iddashboard == dashboardId) { @@ -499,15 +499,12 @@ dashboardMenuList.prepend(items); } - dashboardMenuList.find('a[data-idDashboard]').click(function (e) { + dashboardMenuList.find('a[data-iddashboard]').click(function (e) { e.preventDefault(); - var idDashboard = $(this).attr('data-idDashboard'); + var idDashboard = $(this).attr('data-iddashboard'); - if (typeof piwikMenu != 'undefined') { - piwikMenu.activateMenu('Dashboard', 'embeddedIndex'); - } - $('#Dashboard ul li').removeClass('sfActive'); + $('li', dashboardMenuList).removeClass('sfActive'); if ($(dashboardElement).length) { $(dashboardElement).dashboard('loadDashboard', idDashboard); } else { diff --git a/plugins/Dashboard/templates/index.twig b/plugins/Dashboard/templates/index.twig index ddaac2428c..17f3a5363d 100644 --- a/plugins/Dashboard/templates/index.twig +++ b/plugins/Dashboard/templates/index.twig @@ -6,9 +6,9 @@ <div id="Dashboard" class="piwikTopControl borderedControl piwikSelector"> <ul> {% for dashboard in dashboards %} - <li class="dashboardMenuItem" id="Dashboard_embeddedIndex_{{ dashboard.iddashboard }}"> + <li id="Dashboard_embeddedIndex_{{ dashboard.iddashboard }}"> <a href="javascript:$('#dashboardWidgetsArea').dashboard('loadDashboard', {{ dashboard.iddashboard }});" - class="title">{{ dashboard.name|escape }}</a> + class="item">{{ dashboard.name|escape }}</a> </li> {% endfor %} </ul> |