Skip to content

Commit

Permalink
renderContent not needed in multiselect select inputs.
Browse files Browse the repository at this point in the history
  • Loading branch information
ddooley committed Dec 5, 2023
1 parent b189159 commit b7d2e11
Showing 1 changed file with 4 additions and 10 deletions.
14 changes: 4 additions & 10 deletions lib/DataHarmonizer.js
Original file line number Diff line number Diff line change
Expand Up @@ -1196,11 +1196,7 @@ class DataHarmonizer {
}

$('#field-description-text').html(
`${
fields[col].title
}<select multiple class="multiselect" rows="15">${renderContent(
content
)}</select>`
`${fields[col].title}<select multiple class="multiselect" rows="15">${content}</select>`
);
$('#field-description-modal').modal('show');
$('#field-description-text .multiselect')
Expand All @@ -1213,15 +1209,13 @@ class DataHarmonizer {
let indentation = 12 + label.search(/\S/) * 8; // pixels
return `<div style="padding-left:${indentation}px" class="option ${
value === '' ? 'selectize-dropdown-emptyoptionlabel' : ''
}">${renderContent(escape(label))}</div>`;
}">${escape(label)}</div>`;
},
},
}) // must be rendered when html is visible
.on('change', function () {
let newValCsv = renderContent(
formatMultivaluedValue(
$('#field-description-text .multiselect').val()
)
let newValCsv = formatMultivaluedValue(
$('#field-description-text .multiselect').val()
);
self.setDataAtCell(row, col, newValCsv, 'thisChange');
});
Expand Down

0 comments on commit b7d2e11

Please sign in to comment.