On bar chart selection replace search component with HTML (#12964)

* Sync selected namespace between bar chart and search component

- When user selects the namespace from chart, update the selected
  value in search select component
- Clear out the namespace selection when new date is queried

* On bar chart selection, replace search select with custom html
This commit is contained in:
Arnav Palnitkar 2021-10-29 14:07:00 -07:00 committed by GitHub
parent ab1fd3255b
commit 4d7e946aa6
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 44 additions and 21 deletions

View file

@ -8,6 +8,8 @@ export default class HistoryComponent extends Component {
@tracked selectedNamespace = null; @tracked selectedNamespace = null;
@tracked barChartSelection = false;
// Determine if we have client count data based on the current tab, // Determine if we have client count data based on the current tab,
// since model is slightly different for current month vs history api // since model is slightly different for current month vs history api
get hasClientData() { get hasClientData() {
@ -106,11 +108,17 @@ export default class HistoryComponent extends Component {
// In case of search select component, value returned is an array // In case of search select component, value returned is an array
if (Array.isArray(value)) { if (Array.isArray(value)) {
this.selectedNamespace = this.getNamespace(value[0]); this.selectedNamespace = this.getNamespace(value[0]);
this.barChartSelection = false;
} else if (typeof value === 'object') { } else if (typeof value === 'object') {
// While D3 bar selection returns an object // While D3 bar selection returns an object
this.selectedNamespace = this.getNamespace(value.label); this.selectedNamespace = this.getNamespace(value.label);
} else { this.barChartSelection = true;
}
}
@action
resetData() {
this.barChartSelection = false;
this.selectedNamespace = null; this.selectedNamespace = null;
} }
}
} }

View file

@ -132,7 +132,7 @@
</div> </div>
</div> </div>
{{#if this.showGraphs}} {{#if this.showGraphs}}
<div class="columns has-bottom-margin-m"> <div class="columns has-bottom-margin-m" {{did-update this.resetData}} {{did-insert this.resetData}}>
<div class="column is-two-thirds" data-test-client-count-graph> <div class="column is-two-thirds" data-test-client-count-graph>
<BarChart <BarChart
@title="Top 10 Namespaces" @title="Top 10 Namespaces"
@ -151,6 +151,21 @@
<div class="column"> <div class="column">
<div class="card"> <div class="card">
<div class="card-content"> <div class="card-content">
{{#if (and this.barChartSelection this.selectedNamespace)}}
<label class="title is-5 has-bottom-margin-m">Single namespace</label>
<ul class="has-bottom-margin-l search-select-list">
<li class="search-select-list-item">
<div>
{{or this.selectedNamespace.namespace_path 'root'}}
</div>
<div class="control">
<button type="button" class="button is-ghost" {{action "resetData"}}>
<Icon @glyph="trash" class="has-text-grey" />
</button>
</div>
</li>
</ul>
{{else}}
<SearchSelect <SearchSelect
@id="namespaces" @id="namespaces"
@labelClass="title is-5" @labelClass="title is-5"
@ -161,6 +176,7 @@
@searchField="namespace_path" @searchField="namespace_path"
@selectLimit={{1}} @selectLimit={{1}}
/> />
{{/if}}
{{#if this.selectedNamespace}} {{#if this.selectedNamespace}}
<div class="columns"> <div class="columns">
<div class="column"> <div class="column">

View file

@ -71,7 +71,6 @@ export default Component.extend({
return option.id; return option.id;
}); });
this.set('allOptions', allOptions); // used by filter-wildcard helper this.set('allOptions', allOptions); // used by filter-wildcard helper
let formattedOptions = this.selectedOptions.map(option => { let formattedOptions = this.selectedOptions.map(option => {
let matchingOption = options.findBy('id', option); let matchingOption = options.findBy('id', option);
options.removeObject(matchingOption); options.removeObject(matchingOption);