[SpyPanel] reduced scope of new parameter on visualize, added a default if no option is passed in for show-spy-panel Removed conditional on from discover

This commit is contained in:
Khalah Jones-Golden 2015-10-30 14:51:20 -04:00 committed by Court Ewing
parent bec0201905
commit 3979f0f76c
5 changed files with 9 additions and 5 deletions

View file

@ -28,7 +28,7 @@
<visualize ng-switch-when="visualization"
vis="savedObj.vis"
search-source="savedObj.searchSource"
chrome="chrome"
show-spy-panel="chrome.getVisible()"
class="panel-content">
</visualize>

View file

@ -176,7 +176,7 @@
</header>
<visualize ng-if="vis && rows.length != 0" chrome="chrome" vis="vis" es-resp="mergedEsResp" search-source="searchSource"></visualize>
<visualize ng-if="vis && rows.length != 0" vis="vis" es-resp="mergedEsResp" search-source="searchSource"></visualize>
</div>
<div class="discover-table" fixed-scroll>

View file

@ -138,7 +138,7 @@
</div>
</div>
<visualize chrome="chrome" vis="vis" editable-vis="editableVis" search-source="savedVis.searchSource"></visualize>
<visualize show-spy-panel="chrome.getVisible()" vis="vis" editable-vis="editableVis" search-source="savedVis.searchSource"></visualize>
</div>
</div>

View file

@ -11,4 +11,4 @@
ng-class="{ loading: vis.type.requiresSearch && searchSource.activeFetchCount > 0 }"
class="visualize-chart"></div>
<!-- <pre>{{chartData | json}}</pre> -->
<visualize-spy ng-if="vis.type.requiresSearch && chrome.getVisible()"></visualize-spy>
<visualize-spy ng-if="vis.type.requiresSearch && showSpyPanel"></visualize-spy>

View file

@ -16,7 +16,7 @@ define(function (require) {
return {
restrict: 'E',
scope : {
chrome: '=',
showSpyPanel: '=?',
vis: '=',
editableVis: '=?',
esResp: '=?',
@ -27,6 +27,10 @@ define(function (require) {
var chart; // set in "vis" watcher
var minVisChartHeight = 180;
if (_.isUndefined($scope.showSpyPanel)) {
$scope.showSpyPanel = true;
}
function getter(selector) {
return function () {
var $sel = $el.find(selector);