fix: ui fixes

This commit is contained in:
gosangam
2026-06-13 02:42:07 +05:30
parent 439664a97d
commit 9e68184750
2 changed files with 93 additions and 58 deletions
+44 -17
View File
@@ -2,12 +2,14 @@
/** /**
* CommentMenu — the kebab (⋮) overflow menu on a comment: Pin / Report / * CommentMenu — the kebab (⋮) overflow menu on a comment: Pin / Report /
* Delete (each shown only when permitted). Rendered in a portal with a * Delete (each shown only when permitted). The menu is portaled (fixed
* transparent full-screen backdrop that swallows the dismiss click, so it * position) so it isn't clipped by the comments drawer's overflow, and it
* can't reach (and accidentally close) the surrounding vaul comments drawer. * dismisses via a deferred document `pointerdown`-capture listener + Escape +
* close-on-scroll; the trigger itself toggles. Closing only ever flips this
* menu's own state, so it never closes the surrounding vaul drawer.
*/ */
import { useRef, useState } from "react"; import { useEffect, useRef, useState } from "react";
import { createPortal } from "react-dom"; import { createPortal } from "react-dom";
import { Flag, MoreVertical, Pin, PinOff, Trash2 } from "lucide-react"; import { Flag, MoreVertical, Pin, PinOff, Trash2 } from "lucide-react";
import { cn } from "@/lib/utils"; import { cn } from "@/lib/utils";
@@ -32,11 +34,45 @@ export function CommentMenu({
const [open, setOpen] = useState(false); const [open, setOpen] = useState(false);
const [pos, setPos] = useState<{ top: number; right: number } | null>(null); const [pos, setPos] = useState<{ top: number; right: number } | null>(null);
const btnRef = useRef<HTMLButtonElement>(null); const btnRef = useRef<HTMLButtonElement>(null);
const menuRef = useRef<HTMLDivElement>(null);
// Dismiss on any outside pointer-down, Escape, or scroll. Deferred by a tick
// so the same click that opened the menu doesn't immediately close it.
// Capture phase so it fires regardless of where the target lives in the tree.
useEffect(() => {
if (!open) return;
function onDown(e: PointerEvent) {
const t = e.target as Node;
if (menuRef.current?.contains(t) || btnRef.current?.contains(t)) return;
setOpen(false);
}
function onKey(e: KeyboardEvent) {
if (e.key === "Escape") setOpen(false);
}
function onScroll() {
setOpen(false);
}
const id = window.setTimeout(() => {
document.addEventListener("pointerdown", onDown, true);
document.addEventListener("keydown", onKey);
window.addEventListener("scroll", onScroll, true);
}, 0);
return () => {
window.clearTimeout(id);
document.removeEventListener("pointerdown", onDown, true);
document.removeEventListener("keydown", onKey);
window.removeEventListener("scroll", onScroll, true);
};
}, [open]);
// Nothing actionable → no kebab at all. // Nothing actionable → no kebab at all.
if (!canPin && !canDelete && !canReport) return null; if (!canPin && !canDelete && !canReport) return null;
function openMenu() { function toggle() {
if (open) {
setOpen(false);
return;
}
const r = btnRef.current?.getBoundingClientRect(); const r = btnRef.current?.getBoundingClientRect();
if (r) setPos({ top: r.bottom + 4, right: window.innerWidth - r.right }); if (r) setPos({ top: r.bottom + 4, right: window.innerWidth - r.right });
setOpen(true); setOpen(true);
@@ -47,7 +83,7 @@ export function CommentMenu({
<button <button
ref={btnRef} ref={btnRef}
type="button" type="button"
onClick={openMenu} onClick={toggle}
aria-label="Comment options" aria-label="Comment options"
aria-haspopup="menu" aria-haspopup="menu"
aria-expanded={open} aria-expanded={open}
@@ -57,16 +93,8 @@ export function CommentMenu({
</button> </button>
{open && pos && typeof document !== "undefined" {open && pos && typeof document !== "undefined"
? createPortal( ? createPortal(
<>
<div
className="fixed inset-0 z-[100]"
aria-hidden
onClick={(e) => {
e.stopPropagation();
setOpen(false);
}}
/>
<div <div
ref={menuRef}
role="menu" role="menu"
style={{ position: "fixed", top: pos.top, right: pos.right }} style={{ position: "fixed", top: pos.top, right: pos.right }}
className="z-[101] min-w-40 rounded-lg border border-border bg-card shadow-lg py-1 animate-in fade-in-0 zoom-in-95" className="z-[101] min-w-40 rounded-lg border border-border bg-card shadow-lg py-1 animate-in fade-in-0 zoom-in-95"
@@ -105,8 +133,7 @@ export function CommentMenu({
Delete Delete
</MenuItem> </MenuItem>
) : null} ) : null}
</div> </div>,
</>,
document.body, document.body,
) )
: null} : null}
+10 -2
View File
@@ -127,9 +127,17 @@ export function EmojiPicker({
width: PANEL_WIDTH, width: PANEL_WIDTH,
maxWidth: "calc(100vw - 1rem)", maxWidth: "calc(100vw - 1rem)",
}} }}
className="z-[101] rounded-xl border border-border bg-card shadow-xl p-2 animate-in fade-in-0 zoom-in-95" className="z-[101] flex max-h-[min(60vh,20rem)] flex-col rounded-xl border border-border bg-card shadow-xl animate-in fade-in-0 zoom-in-95"
>
<div
// min-h-0 lets this flex child shrink below its content so it
// actually scrolls; stopPropagation keeps the wheel/touch from
// reaching the host drawer (vaul) and being hijacked.
className="min-h-0 flex-1 overflow-y-auto overscroll-contain p-2"
style={{ WebkitOverflowScrolling: "touch", touchAction: "pan-y" }}
onWheel={(e) => e.stopPropagation()}
onTouchMove={(e) => e.stopPropagation()}
> >
<div className="max-h-60 overflow-y-auto overscroll-contain">
{EMOJI_GROUPS.map((g) => ( {EMOJI_GROUPS.map((g) => (
<div key={g.label} className="mb-1.5"> <div key={g.label} className="mb-1.5">
<p className="px-1 pb-0.5 pt-1 text-[11px] font-medium uppercase tracking-wide text-muted-foreground"> <p className="px-1 pb-0.5 pt-1 text-[11px] font-medium uppercase tracking-wide text-muted-foreground">