Skip to content

Update default group rulers preference and enhance grouped drag functionality#262

Merged
pascalpp merged 1 commit into
pascal/228-feature-multiple-rulersfrom
codex/248-better-group-dragging
Jun 20, 2026
Merged

Update default group rulers preference and enhance grouped drag functionality#262
pascalpp merged 1 commit into
pascal/228-feature-multiple-rulersfrom
codex/248-better-group-dragging

Conversation

@pascalpp

@pascalpp pascalpp commented Jun 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • Improve grouped ruler dragging by temporarily attaching visible follower rulers as child windows during live drags, keeping them visually in sync with the dragged ruler.
  • Refresh follower ruler state after detaching and preserve the existing manual frame-sync fallback for windows that cannot be attached.
  • Change the global Group Rulers preference to default off while preserving persisted true/false values across app launches.
  • Update unit and UI-test expectations for the new default.

Tests

  • xcodebuild -project "Free Ruler.xcodeproj" -scheme "Free Ruler" test -only-testing:FreeRulerTests

Refs #248

…ionality

* Changed the default value of `defaultGroupRulers` to false in `Prefs.swift`.
* Added methods in `RulerManager` to manage the attachment and detachment of grouped drag followers, improving the handling of ruler windows during grouped dragging.
* Updated UI tests and unit tests to reflect the new default behavior for group rulers and ensure proper functionality of grouped dragging.
* Adjusted preferences initialization in `UITestSupport` to align with the new default setting.

pascalpp commented Jun 20, 2026

Copy link
Copy Markdown
Owner Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@pascalpp pascalpp marked this pull request as ready for review June 20, 2026 02:41
@pascalpp pascalpp merged commit b656d6e into pascal/228-feature-multiple-rulers Jun 20, 2026
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