Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transition Region Editor to Tcl/Tk #182

Merged
merged 1 commit into from
Dec 14, 2024
Merged

Conversation

Breakthrough
Copy link
Owner

@Breakthrough Breakthrough commented Dec 5, 2024

Completely changes how the region editor works and operates much more consistently across platforms. Now it has a proper UI, although keyboard is still required for certain tasks. These will be addressed in a follow-up. #181

The benefits are numerous:

  • better integration with system shortcuts for undo/redo
  • more consistent behavior across platforms
  • ability to use mouse to pan/zoom

This also paves the way to transition the region editor into the base of a full DVR-Scan GUI app.

image

@Breakthrough Breakthrough changed the base branch from main to develop December 5, 2024 23:19
@Breakthrough Breakthrough changed the title Rework Region Editor with Tkinter (#181) Rework Region Editor with Tkinter Dec 5, 2024
@Breakthrough Breakthrough force-pushed the issue-181-tk-canvas branch 9 times, most recently from 65b28c8 to b353fb7 Compare December 6, 2024 04:50
@Breakthrough Breakthrough marked this pull request as draft December 6, 2024 15:17
@Breakthrough Breakthrough force-pushed the issue-181-tk-canvas branch 9 times, most recently from ee0dc36 to 2ca27c5 Compare December 7, 2024 23:03
@Breakthrough Breakthrough changed the title Rework Region Editor with Tkinter Transition Region Editor to Tcl/Tk Dec 7, 2024
@Breakthrough Breakthrough force-pushed the issue-181-tk-canvas branch 6 times, most recently from 3990e24 to 053653a Compare December 8, 2024 23:49
@Breakthrough Breakthrough added this to the v1.6.2 milestone Dec 8, 2024
@Breakthrough Breakthrough linked an issue Dec 8, 2024 that may be closed by this pull request
@Breakthrough Breakthrough self-assigned this Dec 8, 2024
@Breakthrough Breakthrough marked this pull request as ready for review December 8, 2024 23:59
This allows for better control over window events and makes input handling
more consistent across platforms. It also allows better customization, for
example, the system undo/redo commands are used instead of key binds.

There is still much work to do, however this is a good starting point, and
has feature parity with the existing editor.
@Breakthrough Breakthrough merged commit 845a5ac into develop Dec 14, 2024
3 of 33 checks passed
@Breakthrough Breakthrough deleted the issue-181-tk-canvas branch December 14, 2024 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Transition Region Editor to Tcl/Tk
1 participant