feat(palette): PROG-123 order move/arc pickers by outline rank with parent hints#97
Merged
Conversation
…arent hints The move picker listed focuses alphabetically with a constant 'Focus' hint, ignoring the manual order the owner curates on the Outline and leaving same-named focuses under different workspaces ambiguous. Focuses now group under their workspace in structure order (sortContainers for workspaces, byRankThenName within), the hint names the parent workspace, and the filter query matches the workspace name since it's on screen. The arc picker likewise sorts byRankThenName and hints the parent focus (the level one up); 'current' still wins on the current arc and 'No arc' stays pinned first. Supersedes PROG-83's alphabetical rule for these two pickers only — docs/decisions/PROG-123.md; REFERENCE picker-order notes updated.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implements PROG-123 — Show parent Workspaces on focus modal switcher.
What
sortContainersfor workspaces,byRankThenNamefor focuses within each), and each row's hint names the parent workspace instead of the constant "Focus". The filter query also matches the workspace name, since it's on screen.Why
PROG-83's "pickers list alphabetically" rule predates manual container ranking — an alphabetical picker fights the order the owner curates on the Outline, and a bare focus name is ambiguous once two workspaces hold same-named focuses. Superseded for these two pickers only (
docs/decisions/PROG-123.md); every other picker stays alphabetical.Verification
bun run check+ 180 unit tests pass; Prettier clean.Follow-up in a second PR (owner review): the flat pickers were replaced by a single Location field + Workspace → Focus → Arc tree picker.