From 27a935a2ff21f48cb296fa7d857f89df3cfe9de9 Mon Sep 17 00:00:00 2001
From: Freeplay <freeplay@duck.com>
Date: Thu, 8 Jun 2023 23:30:14 -0400
Subject: [PATCH 1/4] scrollable sticky widgets attempt

---
 packages/client/src/scripts/sticky-sidebar.ts | 85 +++++++------------
 packages/client/src/ui/universal.vue          | 20 +++--
 packages/client/src/ui/universal.widgets.vue  | 16 ++--
 3 files changed, 48 insertions(+), 73 deletions(-)

diff --git a/packages/client/src/scripts/sticky-sidebar.ts b/packages/client/src/scripts/sticky-sidebar.ts
index 3c42ecb3b..61d39173c 100644
--- a/packages/client/src/scripts/sticky-sidebar.ts
+++ b/packages/client/src/scripts/sticky-sidebar.ts
@@ -2,79 +2,56 @@ export class StickySidebar {
 	private lastScrollTop = 0;
 	private container: HTMLElement;
 	private el: HTMLElement;
-	private spacer: HTMLElement;
-	private marginTop: number;
 	private isTop = false;
 	private isBottom = false;
 	private offsetTop: number;
-	private globalHeaderHeight = 59;
 
 	constructor(
 		container: StickySidebar["container"],
-		marginTop = 0,
-		globalHeaderHeight = 0,
 	) {
 		this.container = container;
+		this.container.style.display = "flex";
 		this.el = this.container.children[0] as HTMLElement;
 		this.el.style.position = "sticky";
-		this.spacer = document.createElement("div");
-		this.container.prepend(this.spacer);
-		this.marginTop = marginTop;
 		this.offsetTop = this.container.getBoundingClientRect().top;
-		this.globalHeaderHeight = globalHeaderHeight;
 	}
 
 	public calc(scrollTop: number) {
-		if (scrollTop > this.lastScrollTop) {
-			// downscroll
-			const overflow = Math.max(
-				0,
-				this.globalHeaderHeight +
-					(this.el.clientHeight + this.marginTop) -
-					window.innerHeight,
-			);
-			this.el.style.bottom = null;
-			this.el.style.top = `${
-				-overflow + this.marginTop + this.globalHeaderHeight
-			}px`;
-
+		if (scrollTop > this.lastScrollTop) { 
+			// downscroll			
+			this.isTop = false;
 			this.isBottom =
-				scrollTop + window.innerHeight >=
+				scrollTop + window.innerHeight >
 				this.el.offsetTop + this.el.clientHeight;
-
-			if (this.isTop) {
-				this.isTop = false;
-				this.spacer.style.marginTop = `${Math.max(
-					0,
-					this.globalHeaderHeight +
-						this.lastScrollTop +
-						this.marginTop -
-						this.offsetTop,
-				)}px`;
-			}
-		} else {
+		} else { 
 			// upscroll
-			const overflow =
-				this.globalHeaderHeight +
-				(this.el.clientHeight + this.marginTop) -
-				window.innerHeight;
-			this.el.style.top = null;
-			this.el.style.bottom = `${-overflow}px`;
+			this.isBottom = false;
+			this.isTop = scrollTop < this.el.offsetTop + 1;
+		}
 
-			this.isTop =
-				scrollTop + this.marginTop + this.globalHeaderHeight <=
-				this.el.offsetTop;
-
-			if (this.isBottom) {
-				this.isBottom = false;
-				this.spacer.style.marginTop = `${
-					this.globalHeaderHeight +
-					this.lastScrollTop +
-					this.marginTop -
-					this.offsetTop -
-					overflow
-				}px`;
+		if (this.isTop) {
+			if (this.el.style.alignSelf != "flex-start") {
+				this.el.style.position = "sticky";
+				this.el.style.bottom = null;
+				this.el.style.top = "0px";
+				this.el.style.alignSelf = "flex-start";
+				console.log("top");
 			}
+			this.offsetTop = scrollTop;
+		} else if (this.isBottom) {
+			if (this.el.style.alignSelf != "flex-end") {
+				this.el.style.position = "sticky";
+				this.el.style.bottom = "0px";
+				this.el.style.top = null;
+				this.el.style.alignSelf = "flex-end";
+				console.log("bottom");
+			}
+			this.offsetTop = window.innerHeight + scrollTop - this.el.scrollHeight;
+		} else {
+			this.el.style.position = "relative";
+			this.el.style.top = this.offsetTop + "px";
+			this.el.style.bottom = null;
+			this.el.style.alignSelf = null;
 		}
 
 		this.lastScrollTop = scrollTop <= 0 ? 0 : scrollTop;
diff --git a/packages/client/src/ui/universal.vue b/packages/client/src/ui/universal.vue
index d376ad1fe..3a7f76708 100644
--- a/packages/client/src/ui/universal.vue
+++ b/packages/client/src/ui/universal.vue
@@ -20,7 +20,7 @@
 		</MkStickyContainer>
 
 		<div v-if="isDesktop" ref="widgetsEl" class="widgets-container">
-			<XWidgets />
+			<XWidgets @mounted="attachSticky" />
 		</div>
 
 		<button
@@ -380,6 +380,13 @@ const onContextmenu = (ev: MouseEvent) => {
 	);
 };
 
+const attachSticky = (el: any) => {
+	const sticky = new StickySidebar(widgetsEl);
+	addEventListener("scroll", () => {
+		sticky.calc(window.scrollY)
+	}, { passive: true });
+};
+
 function top() {
 	window.scroll({ top: 0, behavior: "smooth" });
 }
@@ -440,7 +447,7 @@ console.log(mainRouter.currentRoute.value.name);
 	$widgets-hide-threshold: 1090px;
 
 	// ほんとは単に 100vh と書きたいところだが... https://css-tricks.com/the-trick-to-viewport-units-on-mobile/
-	height: 100%;
+	min-height: calc(var(--vh, 1vh) * 100);
 	box-sizing: border-box;
 	display: flex;
 
@@ -550,14 +557,9 @@ console.log(mainRouter.currentRoute.value.name);
 	}
 
 	> .widgets-container {
-		margin-right: calc(var(--margin) / 2);
-		width: calc(300px + (var(--margin) * 1.5));
-		flex: 0 0 auto;
+		padding: 0 var(--margin);
+		width: 300px;
 		box-sizing: content-box;
-		> :deep(.widgets) {
-			padding-left: var(--margin);
-			padding-right: calc(var(--margin) / 2);
-		}
 
 		@media (max-width: $widgets-hide-threshold) {
 			display: none;
diff --git a/packages/client/src/ui/universal.widgets.vue b/packages/client/src/ui/universal.widgets.vue
index deb5c27eb..6e5f8df23 100644
--- a/packages/client/src/ui/universal.widgets.vue
+++ b/packages/client/src/ui/universal.widgets.vue
@@ -85,22 +85,18 @@ function updateWidgets(widgets) {
 <style lang="scss" scoped>
 .widgets {
 	position: sticky;
-	top: var(--stickyTop, 0px);
 	height: min-content;
 	min-height: 100vh;
-	max-height: 100vh;
-	box-sizing: content-box;
-	overflow: hidden auto;
-	&:not(:hover):not(:focus-within)::-webkit-scrollbar {
-		width: 0;
-	}
+	padding: var(--margin) 0;
+	box-sizing: border-box;
 
 	> * {
 		margin: var(--margin) 0;
 		width: 300px;
-	}
-	> :first-child {
-		margin-top: calc(var(--margin) * 2);
+
+		&:first-child {
+			margin-top: 0;
+		}
 	}
 
 	> .add {

From 5893b4c1177a9c6f3c5a13e0985d2b7861e08c29 Mon Sep 17 00:00:00 2001
From: Freeplay <freeplay@duck.com>
Date: Sat, 10 Jun 2023 16:49:07 -0400
Subject: [PATCH 2/4] simpler way lol

---
 packages/client/src/ui/universal.vue         | 14 +++++++++++---
 packages/client/src/ui/universal.widgets.vue |  1 -
 2 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/packages/client/src/ui/universal.vue b/packages/client/src/ui/universal.vue
index 3a7f76708..4c38648ea 100644
--- a/packages/client/src/ui/universal.vue
+++ b/packages/client/src/ui/universal.vue
@@ -381,9 +381,12 @@ const onContextmenu = (ev: MouseEvent) => {
 };
 
 const attachSticky = (el: any) => {
-	const sticky = new StickySidebar(widgetsEl);
-	addEventListener("scroll", () => {
-		sticky.calc(window.scrollY)
+	let lastScrollTop = 0;
+	addEventListener("scroll", (ev) => {
+		requestAnimationFrame(() => {
+			widgetsEl.scrollTop += window.scrollY - lastScrollTop;
+			lastScrollTop = window.scrollY <= 0 ? 0 : window.scrollY;
+		})
 	}, { passive: true });
 };
 
@@ -557,8 +560,13 @@ console.log(mainRouter.currentRoute.value.name);
 	}
 
 	> .widgets-container {
+		position: sticky;
+		top: 0;
+		max-height: 100vh;
+		overflow-y: auto;
 		padding: 0 var(--margin);
 		width: 300px;
+		min-width: max-content;
 		box-sizing: content-box;
 
 		@media (max-width: $widgets-hide-threshold) {
diff --git a/packages/client/src/ui/universal.widgets.vue b/packages/client/src/ui/universal.widgets.vue
index 6e5f8df23..d931e1bf3 100644
--- a/packages/client/src/ui/universal.widgets.vue
+++ b/packages/client/src/ui/universal.widgets.vue
@@ -84,7 +84,6 @@ function updateWidgets(widgets) {
 
 <style lang="scss" scoped>
 .widgets {
-	position: sticky;
 	height: min-content;
 	min-height: 100vh;
 	padding: var(--margin) 0;

From 2ec1e09105dc3266a26387901d7edb8ffe20cd8c Mon Sep 17 00:00:00 2001
From: Freeplay <freeplay@duck.com>
Date: Sat, 10 Jun 2023 18:08:06 -0400
Subject: [PATCH 3/4] remove scrollbars

---
 packages/client/src/ui/universal.vue | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/packages/client/src/ui/universal.vue b/packages/client/src/ui/universal.vue
index 4c38648ea..a820f145b 100644
--- a/packages/client/src/ui/universal.vue
+++ b/packages/client/src/ui/universal.vue
@@ -568,6 +568,10 @@ console.log(mainRouter.currentRoute.value.name);
 		width: 300px;
 		min-width: max-content;
 		box-sizing: content-box;
+		scrollbar-width: none;
+		&::-webkit-scrollbar {
+			display: none;
+		}
 
 		@media (max-width: $widgets-hide-threshold) {
 			display: none;

From 4bf7357c3073b8adfedea155b103fcceddfeb11f Mon Sep 17 00:00:00 2001
From: Freeplay <freeplay@duck.com>
Date: Sat, 10 Jun 2023 18:33:39 -0400
Subject: [PATCH 4/4] format

---
 packages/client/src/ui/universal.vue | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/packages/client/src/ui/universal.vue b/packages/client/src/ui/universal.vue
index a820f145b..a027cbdd0 100644
--- a/packages/client/src/ui/universal.vue
+++ b/packages/client/src/ui/universal.vue
@@ -382,12 +382,16 @@ const onContextmenu = (ev: MouseEvent) => {
 
 const attachSticky = (el: any) => {
 	let lastScrollTop = 0;
-	addEventListener("scroll", (ev) => {
-		requestAnimationFrame(() => {
-			widgetsEl.scrollTop += window.scrollY - lastScrollTop;
-			lastScrollTop = window.scrollY <= 0 ? 0 : window.scrollY;
-		})
-	}, { passive: true });
+	addEventListener(
+		"scroll",
+		(ev) => {
+			requestAnimationFrame(() => {
+				widgetsEl.scrollTop += window.scrollY - lastScrollTop;
+				lastScrollTop = window.scrollY <= 0 ? 0 : window.scrollY;
+			});
+		},
+		{ passive: true }
+	);
 };
 
 function top() {