orbit/orbit
orbit/orbit
0
0
Fork 0
Code Issues 3 Pull requests 5 Projects Releases Packages Wiki Activity Actions

fix yomirc hover-toolbar clipping (compact toolbar); channel switch always jumps to the newest message

Browse source
This commit is contained in:
Jean Chevronnet 2026-08-01 13:44:05 +00:00
parent dc6ced4dce
commit a51c50d483
No known key found for this signature in database
2 changed files with 9 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/components/chat/MessageList.tsx
View file

@ -76,12 +76,10 @@ export function MessageList() {
const switched = prevActive.current !== active;
prevActive.current = active;
const grew = el.scrollHeight - prevHeight.current;
const d = dividerRef.current;
if (switched && d) {
// Entering a channel that has unread → land on the "New messages" divider so
// you read from where you left off, rather than jumping past it to the bottom.
el.scrollTop = Math.max(0, d.offsetTop - 48);
} else if (switched || atBottom.current) {
// Switching into a channel always jumps to the newest line (the last message),
// even when it has unread — the "New messages" divider still renders as a marker
// to scroll up to. Also follow the tail when already pinned to the bottom.
if (switched || atBottom.current) {
// Buffer switch, or we were pinned to the bottom → follow the newest line.
// atBottom is checked BEFORE the prepend heuristic on purpose: in a short
// buffer the bottom itself sits at scrollTop < 80, so a burst of appended

9
src/themes/yomirc.css
View file

@ -192,10 +192,11 @@
[data-theme^="yomirc"] .mircline--action .mircline__txt { font-style: italic; }
[data-theme^="yomirc"] .mircline.is-redacted { opacity: .55; font-style: italic; }
[data-theme^="yomirc"] .mircline .reactions--inline { display: inline-flex; gap: .25rem; margin-left: .4rem; vertical-align: middle; }
/* Float the hover toolbar ABOVE the dense log line. `top: 0` used to sit it on the
line and 14px INTO the next one — that overlap blocked the next line's hover and
made hovering flicker/jump. Above the line it clears both neighbours. */
[data-theme^="yomirc"] .mircline .msg-actions { top: auto; bottom: calc(100% - 4px); }
/* Compact toolbar that fits WITHIN one dense log line, so it neither clips at the
top of the message list (the old `bottom: calc(100% - 4px)` overhung ~26px above
and got cut) nor overlaps the neighbouring lines' hover. */
[data-theme^="yomirc"] .mircline .msg-actions { top: -1px; bottom: auto; padding: 1px; gap: 0; }
[data-theme^="yomirc"] .mircline .msg-actions button { font-size: .8rem; padding: 1px 3px; }
/* The reply arrow (↩) is a text glyph, not an emoji, so on the pure-black canvas it
falls back to the UA button colour (black) and vanishes. Paint the action glyphs
the classic terminal green so they read against the black. */