Improve modal backdrop scroll prevention and vsizing

This commit is contained in:
Adrian Gruntkowski 2025-03-21 00:20:10 +01:00
parent d68d35e2c1
commit ed4e3ab12d
2 changed files with 21 additions and 8 deletions

View file

@ -166,13 +166,13 @@ editNote.addEventListener("updateNoteInProgress", async (e) => {
editNote.addEventListener("updateNote", async (e) => { editNote.addEventListener("updateNote", async (e) => {
await Notes.update(e.detail); await Notes.update(e.detail);
Notes.saveStorage(); Notes.saveStorage();
editNoteDialog.close(); closeModal(editNoteDialog);
}); });
editNote.addEventListener("deleteNote", async (e) => { editNote.addEventListener("deleteNote", async (e) => {
await Notes.remove(e.detail); await Notes.remove(e.detail);
Notes.saveStorage(); Notes.saveStorage();
editNoteDialog.close(); closeModal(editNoteDialog);
}); });
// The notes rendering routine is optimized to replace only // The notes rendering routine is optimized to replace only
@ -234,6 +234,22 @@ async function render() {
notes.forEach((n) => (currentNotes[n.id] = structuredClone(n))); notes.forEach((n) => (currentNotes[n.id] = structuredClone(n)));
} }
function openModal(modal) {
document.body.style.top = `-${window.scrollY}px`;
document.body.style.position = "fixed";
modal.showModal();
}
function closeModal(modal) {
modal.close();
const scrollY = document.body.style.top;
document.body.style.position = "";
document.body.style.top = "";
window.scrollTo(0, parseInt(scrollY || "0") * -1);
}
function renderNote(note) { function renderNote(note) {
const container = document.createElement("div"); const container = document.createElement("div");
container.id = note.id; container.id = note.id;
@ -287,7 +303,7 @@ function renderNote(note) {
if (e.target.tagName !== "A") { if (e.target.tagName !== "A") {
const note = await Notes.get(container.id); const note = await Notes.get(container.id);
editNote.load(structuredClone(note)); editNote.load(structuredClone(note));
editNoteDialog.showModal(); openModal(editNoteDialog);
} }
}); });

View file

@ -50,7 +50,7 @@ dialog {
border: 0; border: 0;
background: transparent; background: transparent;
max-width: 100vw; max-width: 100vw;
max-height: 100vh; max-height: 90vh;
} }
::backdrop { ::backdrop {
@ -61,10 +61,7 @@ dialog {
body { body {
line-height: 1.5; line-height: 1.5;
-webkit-font-smoothing: antialiased; -webkit-font-smoothing: antialiased;
} width: 100vw;
body:has(dialog[open]) {
overflow: hidden;
} }
img, img,