Something went wrong. Try again.
A fork of helix with a bespoke collection patches.
Something went wrong. Try again.
Fix align_selections command (#15342) master
* Add integration test for bug #15340 This test case handles aligning rows with varying numbers of selections. A bug was found where things would become misaligned in this case. * Fix align_selections command The align selections command wouldn't line up properly when lines had different numbers of selections. The existing code was hard for me to understand with nested vecs and sorting. So I rewrote a large portion of the function body. Now there's two vecs without nested structure, so it should be more efficient and easier to reason about. It also fixes the bug, and rows with varying numbers of selections align.
Author Andrew Davis Committer GitHub Date (Apr 5, 2026, 1:20 PM UTC) Commit d3c238f7 d3c238f7c818bb19fdf1dc90f99ec9ce5b64bd08 Parent 4acbcd82 4acbcd82c4a50ebb58825ac79a62f65337294de1