Skip to content

cleanup: remove dead code and reorganize file structure - #17

Merged
Rubyherp merged 1 commit into
mainfrom
design/drag-size
Jul 13, 2026
Merged

cleanup: remove dead code and reorganize file structure#17
Rubyherp merged 1 commit into
mainfrom
design/drag-size

Conversation

@Rubyherp

@Rubyherp Rubyherp commented Jul 13, 2026

Copy link
Copy Markdown
Owner

Dead code removed:

  • DockPanel: iconSize, containerView, addDragHandle() (drag handle was added via DockDragHandleRepresentable in SwiftUI instead)
  • AppDelegate: stale panel.iconSize assignments, commented-out menu item
  • DockDragHandleView: commented-out calculation
  • DockState: unused items property (items read from @binding instead)

Reorganized:

  • Models/: AppInfo (from Views/), DockState (from root/)
  • Views/: DockDragHandleRepresentable (from Windows/), ScreenRectReader (extracted from TooltipManager.swift)
  • Windows/: DockResizeHandleView (from Views/)
  • Utilities/: new folder, moved Extension.swift from root/

Summary

  • What changed?
  • Why was this change needed?

Testing

  • swift test
  • Manual verification done (if relevant)

Checklist

  • Scope is focused and minimal
  • Docs updated (if needed)
  • No secrets or private data added

Dead code removed:
- DockPanel: iconSize, containerView, addDragHandle() (drag handle
  was added via DockDragHandleRepresentable in SwiftUI instead)
- AppDelegate: stale panel.iconSize assignments, commented-out menu item
- DockDragHandleView: commented-out calculation
- DockState: unused items property (items read from @binding instead)

Reorganized:
- Models/: AppInfo (from Views/), DockState (from root/)
- Views/: DockDragHandleRepresentable (from Windows/), ScreenRectReader
  (extracted from TooltipManager.swift)
- Windows/: DockResizeHandleView (from Views/)
- Utilities/: new folder, moved Extension.swift from root/
@Rubyherp
Rubyherp merged commit 19b2e75 into main Jul 13, 2026
1 check passed
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