mirror of
https://github.com/zadam/trilium.git
synced 2025-01-04 06:13:36 +08:00
fix multivalue issue of not appearing when no attribute has been saved yet
This commit is contained in:
parent
3424406ff1
commit
b0ed790edf
1 changed files with 18 additions and 16 deletions
|
@ -262,25 +262,26 @@ async function loadAttributes() {
|
|||
if (definition.labelType === 'text') {
|
||||
$input.prop("type", "text");
|
||||
|
||||
const attributeValues = await server.get('attributes/values/' + encodeURIComponent(valueAttr.name));
|
||||
// no need to await for this, can be done asynchronously
|
||||
server.get('attributes/values/' + encodeURIComponent(valueAttr.name)).then(attributeValues => {
|
||||
if (attributeValues.length === 0) {
|
||||
return;
|
||||
}
|
||||
|
||||
if (attributeValues.length === 0) {
|
||||
return;
|
||||
}
|
||||
$input.autocomplete({
|
||||
// shouldn't be required and autocomplete should just accept array of strings, but that fails
|
||||
// because we have overriden filter() function in autocomplete.js
|
||||
source: attributeValues.map(attribute => {
|
||||
return {
|
||||
attribute: attribute,
|
||||
value: attribute
|
||||
}
|
||||
}),
|
||||
minLength: 0
|
||||
});
|
||||
|
||||
$input.autocomplete({
|
||||
// shouldn't be required and autocomplete should just accept array of strings, but that fails
|
||||
// because we have overriden filter() function in autocomplete.js
|
||||
source: attributeValues.map(attribute => {
|
||||
return {
|
||||
attribute: attribute,
|
||||
value: attribute
|
||||
}
|
||||
}),
|
||||
minLength: 0
|
||||
$input.focus(() => $input.autocomplete("search", ""));
|
||||
});
|
||||
|
||||
$input.focus(() => $input.autocomplete("search", ""));
|
||||
}
|
||||
else if (definition.labelType === 'number') {
|
||||
$input.prop("type", "number");
|
||||
|
@ -352,6 +353,7 @@ async function loadAttributes() {
|
|||
|
||||
$multiplicityCell.append(addButton).append(" ").append(removeButton);
|
||||
}
|
||||
|
||||
return $tr;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue