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

The printing function in the PZero terminal is not implemented correctly. #122

Open
r-monti14 opened this issue Dec 2, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@r-monti14
Copy link
Collaborator

With the projections tools, TextTerminal is not working and it freeze the GUI.
In addition, some messages must be review: for istance, when usinig split line-line, the message is "Split line with line. Line to be split has been selected, please select an intersecting line", but to use this tool it is necessary to select the lines to be split and the intersection line before activating the tool.

@r-monti14
Copy link
Collaborator Author

@andrea-bistacchi I saw that you changed some lines in https://github.com/gecos-lab/PZero/blob/class_refactoring_pyside6/pzero/two_d_lines.py moving from self.parent.TextTerminal.appendPlainText() to self.print_terminal().
I test it also in https://github.com/gecos-lab/PZero/blob/class_refactoring_pyside6/pzero/three_d_surfaces.py and seems to be a nice solution. Should I homogenise it in both files?

@r-monti14
Copy link
Collaborator Author

Fixed with 57d72d8
We should check if TextTerminal is used in other files

@r-monti14 r-monti14 added the enhancement New feature or request label Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant