Skip to content

Feature/editor improvements#68

Open
maciekswat wants to merge 5 commits into
developfrom
feature/editor-improvements
Open

Feature/editor improvements#68
maciekswat wants to merge 5 commits into
developfrom
feature/editor-improvements

Conversation

@maciekswat

Copy link
Copy Markdown
Collaborator

Few Improvements

  1. added automatic closing of brackets, and quotes so that when you type you don't need to type ( and then ) you just type ( and closing bracket is displayed automatically and cursor is positioned in the correct place. This option in ON by default
  2. Added auto-save - ON by default
  3. restored missing configuration.ui and fixed persistence of boolean settings - OFF settings were not persisted
    Here is summary of the new settings dialog and it also whos new features - auto-match characters and autosave
image

@maciekswat maciekswat requested review from bartjuw and fyffep June 28, 2026 02:00

@fyffep fyffep left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Aside from the concern about auto-save, this looks good. I researched a few things about how Qscintilla handles key events.


self.defaultConfigs["EnableAutoSave"] = True

self.defaultConfigs["AutoSaveInterval"] = 1000

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This may cause a performance degradation or a stuttering feeling while typing. I recommend changing it.

Options:

  1. Change the default to every 1 minute. While 5-10 minutes is more common in other software, Twedit mainly operates on small text files.
  2. Make Twedit autosave after every keystroke instead of using a time-based saver.

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.

2 participants