BBRUNER PatrickFix scan-token race and screen-size-dependent layout test
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
file namespace | 1 年前 | |
Add hide-line flag and Core line visibility map/tracker (#338) HighlightEntry.IsHideLine (default false, cloned, serialized), HighlightEvaluator.IsHidden, an immutable row<->original-line map with append-only shared storage, and a tracker that rebuilds the map on a cancellable background scan with generation-tagged results. | 19 天前 | |
Address second review of hide-line PR (#338) - Queue explicit navigation made before the first scan as the call itself, keeping its time-sync flag and follow-tail state; timestamp navigation from the sync origin is queued too and reports it did not scroll - Drop queued navigation when a reload replaces loaded content - Restoring a saved or reload position selects through SelectRow again (clears a pending time sync, focuses the grid) - VisibleRows takes an IVisibleRowsHost instead of three callbacks and owns the current-line lookup - Rename NavigationRow to RevealOrNearestRow; the pin delegate is required; comment fixes - Tests: bookmark window and timestamp navigation during the first scan | 18 天前 | |
removed ConfigManager and Preferences from PaintHelper | 1 年前 | |
Show no rows until the first hide scan; time-sync followers stay hidden (#338) - While a load's first visibility scan runs the grid shows no rows, and saved/reload positions are applied once it has published. - Windows following a time sync select the nearest visible line; only the window the user navigates in reveals a hidden line. - Review fixes: shared evaluate-and-commit loop in the tracker, one publish path for the grid, CurrentLineNum instead of a duplicate property, line/row naming, unsubscribe on close, drop unused Truncate. | 19 天前 | |
Fix scan-token race and screen-size-dependent layout test LineVisibilityTracker read the scan's cancellation token inside the queued task; disposing or restarting before the task started faulted it with ObjectDisposedException. Read the token under the lock instead. HighlightDialogLayoutTests assumed the scaled minimum size fits the screen; Windows caps forms at the screen size, so 1.5x/2x failed on the 1024x768 CI runner. Treat that case as inconclusive. Name groupBoxSelection so overlap messages identify it. | 12 天前 |