Skip to content

DndHandler & AbstractDirectoryView cleanup#2792

Merged
lenemter merged 5 commits into
mainfrom
jeremypw/dnd/clean-up
May 14, 2026
Merged

DndHandler & AbstractDirectoryView cleanup#2792
lenemter merged 5 commits into
mainfrom
jeremypw/dnd/clean-up

Conversation

@jeremypw
Copy link
Copy Markdown
Contributor

@jeremypw jeremypw commented May 9, 2026

Extracted from #2771

  • Make all member and property scopes explicit in AbstractDirectory View
  • Make format of method parameter indentation consistent in DnDHandler

No functional changes.

@jeremypw jeremypw requested a review from a team May 9, 2026 19:02
Comment thread src/View/AbstractDirectoryView.vala Outdated
Comment thread src/View/AbstractDirectoryView.vala Outdated
Comment thread src/View/AbstractDirectoryView.vala Outdated
Comment thread src/View/AbstractDirectoryView.vala Outdated
@jeremypw
Copy link
Copy Markdown
Contributor Author

@lenemter Thanks for spotting the indentation errors - now fixed.

Copy link
Copy Markdown
Member

@danirabbit danirabbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

Comment thread src/View/AbstractDirectoryView.vala
@lenemter lenemter force-pushed the jeremypw/dnd/clean-up branch from fbf26cb to ff46076 Compare May 14, 2026 16:32
@lenemter lenemter enabled auto-merge (squash) May 14, 2026 16:32
@lenemter lenemter merged commit a3d5863 into main May 14, 2026
4 checks passed
@lenemter lenemter deleted the jeremypw/dnd/clean-up branch May 14, 2026 16:33
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.

3 participants