mirrors/hermes-agent
1
0
Fork 0
mirror of https://github.com/NousResearch/hermes-agent.git synced 2026-05-08 03:01:47 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

fix(tui): keep selection drag responsive

Browse source
This commit is contained in:
Brooklyn Nicholson 2026-04-26 04:44:19 -05:00
parent 7d68ea9501
commit e16e196c7e
1 changed files with 22 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

37
ui-tui/packages/hermes-ink/src/ink/ink.tsx
View file

@ -65,6 +65,7 @@ import {
type SelectionState, type SelectionState,
selectLineAt, selectLineAt,
selectWordAt, selectWordAt,
selectionBounds,
shiftAnchor, shiftAnchor,
shiftSelection, shiftSelection,
shiftSelectionForFollow, shiftSelectionForFollow,
@ -252,7 +253,6 @@ export default class Ink {
// into one follow-up microtask instead of stacking renders. // into one follow-up microtask instead of stacking renders.
private isRendering = false private isRendering = false
private immediateRerenderRequested = false private immediateRerenderRequested = false
private selectionNotifyQueued = false
private selectionDragCell: { col: number; row: number } | null = null private selectionDragCell: { col: number; row: number } | null = null
private selectionAutoScrollTimer: ReturnType<typeof setInterval> | null = null private selectionAutoScrollTimer: ReturnType<typeof setInterval> | null = null
private selectionAutoScrollDir: -1 | 0 | 1 = 0 private selectionAutoScrollDir: -1 | 0 | 1 = 0
@ -1606,13 +1606,7 @@ export default class Ink {
return () => this.selectionListeners.delete(cb) return () => this.selectionListeners.delete(cb)
} }
private notifySelectionChange(): void { private notifySelectionChange(): void {
if (!this.selectionNotifyQueued) { this.scheduleRender()
this.selectionNotifyQueued = true
queueMicrotask(() => {
this.selectionNotifyQueued = false
this.scheduleRender()
})
}
const active = hasSelection(this.selection) const active = hasSelection(this.selection)
@ -1854,11 +1848,16 @@ export default class Ink {
return return
} }
if (this.selectionAutoScrollDir > 0) { const top = box.scrollViewportTop ?? 0
captureScrolledRows(this.selection, this.frontFrame.screen, box.scrollViewportTop ?? 0, box.scrollViewportTop ?? 0, 'above') const bottom = top + viewport - 1
} else { const before = selectionBounds(this.selection)
const bottom = (box.scrollViewportTop ?? 0) + viewport - 1
captureScrolledRows(this.selection, this.frontFrame.screen, bottom, bottom, 'below') if (before) {
if (this.selectionAutoScrollDir > 0) {
captureScrolledRows(this.selection, this.frontFrame.screen, top, top, 'above')
} else {
captureScrolledRows(this.selection, this.frontFrame.screen, bottom, bottom, 'below')
}
} }
box.stickyScroll = false box.stickyScroll = false
@ -1866,8 +1865,16 @@ export default class Ink {
box.scrollAnchor = undefined box.scrollAnchor = undefined
box.scrollTop = next box.scrollTop = next
markDirty(box) markDirty(box)
shiftAnchor(this.selection, -this.selectionAutoScrollDir, box.scrollViewportTop ?? 0, (box.scrollViewportTop ?? 0) + viewport - 1) shiftAnchor(this.selection, -this.selectionAutoScrollDir, top, bottom)
this.applySelectionDrag(this.selectionDragCell?.col ?? 0, this.selectionAutoScrollDir > 0 ? this.terminalRows - 1 : 0)
if (this.selectionDragCell) {
this.selectionDragCell = {
col: this.selectionDragCell.col,
row: this.selectionAutoScrollDir > 0 ? bottom : top
}
}
this.applySelectionDrag(this.selectionDragCell?.col ?? 0, this.selectionDragCell?.row ?? (this.selectionAutoScrollDir > 0 ? bottom : top))
} }
private stopSelectionAutoScroll(): void { private stopSelectionAutoScroll(): void {