diff options
author | Ben Burgess <88810029+bx80@users.noreply.github.com> | 2021-12-06 16:36:19 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-12-06 16:36:19 +0300 |
commit | b7bfb36a3d37e4d056ab3f0355672b85f318f3d9 (patch) | |
tree | 9bacb30a15f353eed13fff305443d9bcaa9fcf41 /plugins/CoreHome | |
parent | 9d2ba3ad410ae07cf2021de034637f6bd11b8fc0 (diff) |
Rendering performance improvement for reports with many rows (#18405)
* Remove obsolete datatable handleColumnHighlighting method
* Restore handleColumnHighlighting method but without width setting code
* Replace datatable setFixWidthToMakeEllipsisWork() method with CSS
* Restore datatable setFixWidthToMakeEllipsisWork() method
* Updated UI screenshots
* Update submodule
* update submodule
* update submodule
Co-authored-by: Stefan Giehl <stefan@matomo.org>
Diffstat (limited to 'plugins/CoreHome')
-rw-r--r-- | plugins/CoreHome/javascripts/dataTable.js | 27 |
1 files changed, 1 insertions, 26 deletions
diff --git a/plugins/CoreHome/javascripts/dataTable.js b/plugins/CoreHome/javascripts/dataTable.js index 1453f257b9..76859d7408 100644 --- a/plugins/CoreHome/javascripts/dataTable.js +++ b/plugins/CoreHome/javascripts/dataTable.js @@ -1431,35 +1431,10 @@ $.extend(DataTable.prototype, UIControl.prototype, { }, handleColumnHighlighting: function (domElem) { - var maxWidth = {}; + var currentNthChild = null; var self = this; - // give all values consistent width - $('td', domElem).each(function () { - var $this = $(this); - if ($this.hasClass('label')) { - return; - } - - var table = $this.closest('table'); - var nthChild = $this.parent('tr').children().index($(this)) + 1; - var rows = $('> tbody > tr', table); - - if (!maxWidth[nthChild]) { - maxWidth[nthChild] = 0; - rows.find("td:nth-child(" + (nthChild) + ").column .value").add('> thead th:not(.label) .thDIV', table).each(function (index, element) { - var width = $(element).width(); - if (width > maxWidth[nthChild]) { - maxWidth[nthChild] = width; - } - }); - rows.find("td:nth-child(" + (nthChild) + ").column .value").each(function (index, element) { - $(element).closest('td').css({width: maxWidth[nthChild]}); - }); - } - }); - // highlight all columns on hover $(domElem).on('mouseenter', 'td', function (e) { e.stopPropagation(); |