Etheryo/curriculum-vitae
1
0
Fork 0
Code Issues 2 Pull requests Projects Releases 1 Packages Wiki Activity Actions

Frontend: Fixing popup not showing if the button is not aligned with the popup (calling leavingPopup prematurly).

Browse source
This commit is contained in:
Yohan Boujon 2024-01-30 23:25:41 +01:00
parent 929158a32d
commit f8ec7e427d
2 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
frontend/src/lib/components/projects-popup.svelte
View file

@ -64,7 +64,9 @@
application_link = data.application_link; application_link = data.application_link;
filteredTags = filterTag(tags, id); filteredTags = filterTag(tags, id);
// Active set to true after the await to avoid conflict when clicking outside while the popup hasn't showed yet. // Active set to true after the await to avoid conflict when clicking outside while the popup hasn't showed yet.
setTimeout(() => {
active = true; active = true;
}, 10);
} }
} }
@ -73,7 +75,7 @@
}); });
function hidePopup(event) { function hidePopup(event) {
if (!active) { if (!active && popupMain.style.visibility === "visible") {
return; return;
} }

1
frontend/src/routes/+page.svelte
View file

@ -51,7 +51,6 @@
let isBottom = scrollY + innerHeight >= footer.offsetTop; let isBottom = scrollY + innerHeight >= footer.offsetTop;
let isMoved = scrollY + innerHeight >= sidebar.offsetHeight; let isMoved = scrollY + innerHeight >= sidebar.offsetHeight;
let littleScreen = innerHeight < sidebar.offsetHeight; let littleScreen = innerHeight < sidebar.offsetHeight;
console.log(littleScreen);
// Only having the sticky sidebar if the size of the screen is too 'little' // Only having the sticky sidebar if the size of the screen is too 'little'
// Testing if sidebar is outside of scrolling scope // Testing if sidebar is outside of scrolling scope