From 6d0bd75cb61860d97698f4d5b04ad7b785c00d0d Mon Sep 17 00:00:00 2001
From: hypercross <helios.tnak@gmail.com>
Date: Sat, 11 Jul 2026 09:49:48 +0800
Subject: [PATCH] refactor: simplify button positioning in RevealManager

Remove unused scroll container dependency and use relative rect
coordinates for button positioning.
---
 src/components/RevealManager.tsx | 14 ++------------
 1 file changed, 2 insertions(+), 12 deletions(-)

diff --git a/src/components/RevealManager.tsx b/src/components/RevealManager.tsx
index e5e8857..490cdd3 100644
--- a/src/components/RevealManager.tsx
+++ b/src/components/RevealManager.tsx
@@ -18,7 +18,6 @@ import {
 import { Portal } from "solid-js/web";
 import { useLocation } from "@solidjs/router";
 import { useArticleDom } from "./Article";
-import { useScrollContainer } from "../App";
 import { journalStreamState } from "./stores/journalStream";
 import { useJournalCompletions } from "./journal/completions";
 import {
@@ -76,7 +75,6 @@ function hide() {
 
 export const RevealManager: Component = () => {
   const contentDom = useArticleDom();
-  const scrollContainer = useScrollContainer();
   const location = useLocation();
   const comp = useJournalCompletions();
 
@@ -181,16 +179,8 @@ export const RevealManager: Component = () => {
           <button
             class={`fixed z-50 inline-flex items-center justify-center w-5 h-5 rounded cursor-pointer ${btn().color}`}
             style={{
-              top: `${
-                btn().rect.top +
-                (scrollContainer()?.scrollTop ?? window.scrollY) -
-                6
-              }px`,
-              left: `${
-                btn().rect.left +
-                (scrollContainer()?.scrollLeft ?? window.scrollX) -
-                20
-              }px`,
+              top: `${btn().rect.top - 6}px`,
+              left: `${btn().rect.left - 20}px`,
             }}
             title={btn().title}
             innerHTML={btn().svg}