small fixes for collapse/expand

This commit is contained in:
zadam 2020-05-03 13:52:12 +02:00
parent 2a3091f788
commit 43e12fbea2
4 changed files with 13 additions and 6 deletions

View File

@ -231,7 +231,7 @@ class TreeCache {
/** @return {Promise<NoteShort>} */
async getNote(noteId, silentNotFoundError = false) {
if (noteId === 'none') {
console.log(`No 'none' note.`);
console.trace(`No 'none' note.`);
return null;
}
else if (!noteId) {

View File

@ -40,9 +40,9 @@ class TreeContextMenu {
async getMenuItems() {
const note = await treeCache.getNote(this.node.data.noteId);
const branch = treeCache.getBranch(this.node.data.branchId);
const parentNote = await treeCache.getNote(branch.parentNoteId);
const isNotRoot = note.noteId !== 'root';
const isHoisted = note.noteId === hoistedNoteService.getHoistedNoteId();
const parentNote = isNotRoot ? await treeCache.getNote(branch.parentNoteId) : null;
// some actions don't support multi-note so they are disabled when notes are selected
// the only exception is when the only selected note is the one that was right-clicked, then

View File

@ -606,7 +606,9 @@ export default class NoteTreeWidget extends TabAwareWidget {
await this.batchUpdate(async () => {
await node.load(true);
await node.setExpanded(isExpanded, {noEvents: true});
if (node.data.noteId !== 'root') { // root is always expanded
await node.setExpanded(isExpanded, {noEvents: true});
}
});
}

View File

@ -114,14 +114,16 @@ async function moveBranchAfterNote(req) {
async function setExpanded(req) {
const {branchId, expanded} = req.params;
await sql.execute("UPDATE branches SET isExpanded = ? WHERE branchId = ?", [expanded, branchId]);
// we don't sync expanded label
if (branchId !== 'root') {
await sql.execute("UPDATE branches SET isExpanded = ? WHERE branchId = ?", [expanded, branchId]);
// we don't sync expanded label
}
}
async function setExpandedForSubtree(req) {
const {branchId, expanded} = req.params;
const branchIds = await sql.getColumn(`
let branchIds = await sql.getColumn(`
WITH RECURSIVE
tree(branchId, noteId) AS (
SELECT branchId, noteId FROM branches WHERE branchId = ?
@ -132,6 +134,9 @@ async function setExpandedForSubtree(req) {
)
SELECT branchId FROM tree`, [branchId]);
// root is always expanded
branchIds = branchIds.filter(branchId => branchId !== 'root');
await sql.executeMany(`UPDATE branches SET isExpanded = ${expanded} WHERE branchId IN (???)`, branchIds);
return {