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

UXUI3: Options, Find&Replace, 3D G-Code Viewer #1367

Merged
merged 9 commits into from
Aug 14, 2023

Conversation

neofelis2X
Copy link
Contributor

@neofelis2X neofelis2X commented Jul 20, 2023

Hello,
this is Pronterface ui tweaking part 3. It contains the following changes and enhancements:

  • Refactored get_space function. It is now much more elegant and does not rely on Python 3.10 features. Fixes Pronterface broken in Python 3.9 or older #1362

  • Macro Editor: 'Find and replace' dialogue is now complete with all options working
    search

  • Options dialogue: some additional formatting, widgets now have homogeneous lengths
    settings

  • Options dialogue: Added new setting type for directories, for cases like 'Log path'. They now have a 'Browse' button which opens a 'Choose Directory...' dialogue
    browse

  • G-Code Viewer 3D view: An oversight on my part, sizer and toolbar work correctly now in 3D mode.

  • G-Code Viewer 3D view: The background colour of the 3D view is now the same as in the main window.
    gcode3d

  • utils.py: just linting... I started to implement something here, but ended up not doing it and just keeping the cleanup changes.

As always, don't spare the feedback! :) And have a nice summer.

Copy link
Collaborator

@rockstorm101 rockstorm101 left a comment

Choose a reason for hiding this comment

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

Sorry it took me this long to review this. Good job here.

printrun/utils.py Outdated Show resolved Hide resolved
printrun/utils.py Show resolved Hide resolved
@rockstorm101 rockstorm101 added this to the v2.1 milestone Aug 13, 2023
@rockstorm101
Copy link
Collaborator

Can't thank you enough for these contributions. Much appreciated.

Out of curiosity, are you planning on more UX updates? If yes, awesome! keep them coming! If not, great as well, I think it is time for a new release then :)

@rockstorm101 rockstorm101 merged commit 2f2b7d7 into kliment:master Aug 14, 2023
21 checks passed
@neofelis2X
Copy link
Contributor Author

Great, thank you for reviewing it @rockstorm101 !

Out of curiosity, are you planning on more UX updates?

Yes I have plenty of points that I would like to address. Some are already in the making. I've got a few more updates to make to the dialogues, and then maybe we can draw the line for a new release in autumn.

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.

Pronterface broken in Python 3.9 or older
2 participants