Skip to content

Cleanup: Simplify open document#1773

Draft
jeremypw wants to merge 7 commits into
masterfrom
jeremypw/open-document/simplify
Draft

Cleanup: Simplify open document#1773
jeremypw wants to merge 7 commits into
masterfrom
jeremypw/open-document/simplify

Conversation

@jeremypw

@jeremypw jeremypw commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator
  • Move creation of Document objects into DocumentView and change first parameter to string (path)
  • Only create new objects when needed
  • Update calls to this function appropriately
  • Lose unneeded intermediate functions in MainWindow and call the Document View function directly
  • Simplify the open_document function by DRYing and treating all documents the same regarding focus, cursor_position and range, whether newly created or existing .

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