This is an automated email from the ASF dual-hosted git repository.
rohit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git
The following commit(s) were added to refs/heads/master by this push:
new 29125be ui: Fix quick view tooltip title on multiselect list views
(#3403)
29125be is described below
commit 29125be369ed845168fc0a5941c2dafdb3913646
Author: Nico Wohlfarth <n.wohlfa...@ewerk.com>
AuthorDate: Thu Jun 13 12:23:29 2019 +0200
ui: Fix quick view tooltip title on multiselect list views (#3403)
Fixed an issue, where the quick view tooltip title would not have been
displayed properly on list views with a multiselect row.
This was because the html content of the first td of a row was used to
render the title of the quick view tooltip, which would not work if the first
columns content was a checkbox.
Now the td with class first will be used for this (this will be the second
column if there is a multiselect column, otherwise it will remain the first
column).
---
ui/scripts/ui/widgets/listView.js | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/ui/scripts/ui/widgets/listView.js
b/ui/scripts/ui/widgets/listView.js
index 52f3d7b..7162b93 100644
--- a/ui/scripts/ui/widgets/listView.js
+++ b/ui/scripts/ui/widgets/listView.js
@@ -1585,10 +1585,10 @@
$('<span>').html(_l('label.quickview') + ': '),
$('<span>').addClass('title').html(
cloudStack.concat(
- $tr.find('td:first span').html(), 30
+ $tr.find('td.first span').html(), 30
)
).attr({
- title: $tr.find('td:first span').html()
+ title: $tr.find('td.first span').html()
}),
$('<span>').addClass('icon').html(' ')
);
