openstack/horizon
Code Issues Proposed changes

Relax hz-table parent requirement

Browse Source 
hz-expand-detail is a pretty handy directive to use in normal
HTML tables. This patch changes the parent directive requirement
to be optional.

Change-Id: Ic0b0c7af6143ec1d654cb277503342693d051b50
Closes-Bug: #1580809
This commit is contained in:
Tyr Johanson 2016-05-25 09:49:06 -06:00 committed by Matt Borland
parent b0d1c88bbd
commit fe98059ca1
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
horizon/static/framework/widgets/table/hz-expand-detail.directive.js
View File

@ -59,7 +59,7 @@
function hzExpandDetail(settings) { function hzExpandDetail(settings) {
var directive = { var directive = {
restrict: 'A', restrict: 'A',
require: '^hzTable', require: '?^hzTable',
scope: { scope: {
icons: '@hzExpandDetail', icons: '@hzExpandDetail',
duration: '@', duration: '@',
@ -101,7 +101,7 @@
detailCell.wrapInner('<div class="detail-expanded"></div>'); detailCell.wrapInner('<div class="detail-expanded"></div>');
} }
if (scope.item) { if (scope.item && hzTableCtrl) {
hzTableCtrl.broadcastExpansion(scope.item); hzTableCtrl.broadcastExpansion(scope.item);
} }