refactoring
This commit is contained in:
@@ -2,7 +2,7 @@ class BatchCodeDetailsView {
|
||||
|
||||
#uiContainer;
|
||||
#headingElement;
|
||||
#batchcodesSelectElement;
|
||||
#batchCodesSelectElement;
|
||||
#adverseReactionReportsChartView;
|
||||
#histogramChartView;
|
||||
#chartWithSlider;
|
||||
@@ -10,7 +10,7 @@ class BatchCodeDetailsView {
|
||||
constructor(uiContainer) {
|
||||
this.#uiContainer = uiContainer
|
||||
this.#headingElement = this.#uiContainer.querySelector(".heading");
|
||||
this.#batchcodesSelectElement = new ElementWithSingleChangeEventListener(this.#uiContainer.querySelector("#batchcodesSelect"));
|
||||
this.#batchCodesSelectElement = new ElementWithSingleChangeEventListener(this.#uiContainer.querySelector("#batchCodesSelect"));
|
||||
this.#adverseReactionReportsChartView = new AdverseReactionReportsChartView(this.#uiContainer.querySelector('#adverseReactionReportsChartView'));
|
||||
this.#chartWithSlider = this.#uiContainer.querySelector('.chartWithSlider');
|
||||
this.#histogramChartView = new HistogramChartView(this.#chartWithSlider.querySelector("canvas"));
|
||||
@@ -58,7 +58,7 @@ class BatchCodeDetailsView {
|
||||
#displaySelectBatchcodeCombination(histograms) {
|
||||
BatchcodeCombinationSelection.configureSelectBatchcodeCombinationElement(
|
||||
{
|
||||
batchcodesSelectElement: this.#batchcodesSelectElement,
|
||||
batchCodesSelectElement: this.#batchCodesSelectElement,
|
||||
histograms: histograms,
|
||||
onSelect: histoDescr => this.#displayHistogram(histoDescr)
|
||||
});
|
||||
|
||||
@@ -1,8 +1,8 @@
|
||||
class BatchcodeCombinationSelection {
|
||||
|
||||
static configureSelectBatchcodeCombinationElement({ batchcodesSelectElement, histograms, onSelect }) {
|
||||
this.#setBatchcodeCombinationOptions(batchcodesSelectElement.element, histograms);
|
||||
batchcodesSelectElement.setSingleChangeEventListener(
|
||||
static configureSelectBatchcodeCombinationElement({ batchCodesSelectElement, histograms, onSelect }) {
|
||||
this.#setBatchcodeCombinationOptions(batchCodesSelectElement.element, histograms);
|
||||
batchCodesSelectElement.setSingleChangeEventListener(
|
||||
event => {
|
||||
const histoDescr = histograms[event.target.value];
|
||||
onSelect(histoDescr);
|
||||
@@ -10,9 +10,9 @@ class BatchcodeCombinationSelection {
|
||||
onSelect(histograms[0]);
|
||||
}
|
||||
|
||||
static #setBatchcodeCombinationOptions(batchcodesSelectElement, histograms) {
|
||||
UIUtils.clear(batchcodesSelectElement);
|
||||
this.#getBatchcodeCombinationOptions(histograms).forEach(option => batchcodesSelectElement.add(option));
|
||||
static #setBatchcodeCombinationOptions(batchCodesSelectElement, histograms) {
|
||||
UIUtils.clear(batchCodesSelectElement);
|
||||
this.#getBatchcodeCombinationOptions(histograms).forEach(option => batchCodesSelectElement.add(option));
|
||||
}
|
||||
|
||||
static #getBatchcodeCombinationOptions(histograms) {
|
||||
|
||||
@@ -49768,7 +49768,7 @@
|
||||
<canvas id="adverseReactionReportsChartView"></canvas>
|
||||
<div id="selectBatchcodeCombination">
|
||||
<label>Select batchcode combination:
|
||||
<select id="batchcodesSelect" name="batchcodes">
|
||||
<select id="batchCodesSelect" name="batchcodes">
|
||||
</select>
|
||||
</label>
|
||||
<div class="chartWithSlider">
|
||||
|
||||
Reference in New Issue
Block a user