Skip to content

Allow reordering non-name table columns - #2

Merged
lyc280705 merged 1 commit into
mainfrom
agent/fix-column-reordering
Jul 10, 2026
Merged

Allow reordering non-name table columns#2
lyc280705 merged 1 commit into
mainfrom
agent/fix-column-reordering

Conversation

@lyc280705

Copy link
Copy Markdown
Owner

What changed

  • allow Type, Size, Modified, and Location columns to be freely reordered
  • keep the Name column fixed at the leading edge
  • restore the Name column after a non-name column is dropped at index zero

Root cause

AppKit first calls tableView(_:shouldReorderColumn:toColumn:) with a target index of -1 when a drag begins. The previous destination-index check returned false at that stage, which disabled the entire drag.

Validation

  • confirmed the documented initial -1 callback behavior in Apple's NSTableViewDelegate documentation
  • built the universal app and Finder service locally
  • checked the focused source diff and whitespace

@lyc280705
lyc280705 marked this pull request as ready for review July 10, 2026 07:52
@lyc280705
lyc280705 merged commit c75a729 into main Jul 10, 2026
1 check passed
@lyc280705
lyc280705 deleted the agent/fix-column-reordering branch July 10, 2026 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant