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

gui/func: Add a 'delete after conversion' button (closes #458) #488

Merged
merged 3 commits into from
Mar 2, 2023

Conversation

Constantin1489
Copy link
Contributor

@Constantin1489 Constantin1489 commented Mar 2, 2023

added 'Delete after conversion' button.
I think the button works but in the full scenario, I'm not sure.
I just imitated other button objects like self.croppingBox.
So please check the code before merging.

@Constantin1489 Constantin1489 changed the title gui/func: add delete button (closes ciromattia#458) gui/func: Add delete button (closes ciromattia#458) Mar 2, 2023
@@ -294,6 +294,10 @@ def run(self):
options.maximizestrips = True
if GUI.disableProcessingBox.isChecked():
options.noprocessing = True
if GUI.deleteBox.checkState() == 1:
options.delete = 2
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure this part. options.delete = 2 and so on.

@Constantin1489 Constantin1489 changed the title gui/func: Add delete button (closes ciromattia#458) gui/func: Add a 'delete after conversion' button (closes ciromattia#458) Mar 2, 2023
@Constantin1489 Constantin1489 changed the title gui/func: Add a 'delete after conversion' button (closes ciromattia#458) gui/func: Add a 'delete after conversion' button (closes #458) Mar 2, 2023
@darodi
Copy link
Collaborator

darodi commented Mar 2, 2023

@Constantin1489

Thanks for your PR.

You do not need a tristate in your case and can use

  if GUI.deleteBox.isChecked():
            options.delete= True

instead

I'll try to do the modification when I find some time.

@darodi darodi merged commit 17206dd into ciromattia:master Mar 2, 2023
@darodi darodi added the Enhancement New feature or request label Mar 2, 2023
@darodi darodi added this to the 5.6.2 milestone Mar 2, 2023
@Constantin1489
Copy link
Contributor Author

Constantin1489 commented Mar 7, 2023

@darodi Thank you for code reviewing and merging. This was my first experience committing to open-source. I appreciate collaborating with you.

darodi pushed a commit to darodi/kcc that referenced this pull request Mar 27, 2023
* gui/func: add delete button (closes ciromattia#458)

* Update KCC_ui.py

fix typo

* GUI delete option checkBox

---------

Co-authored-by: darodi <4682830+darodi@users.noreply.github.com>
(cherry picked from commit 17206dd)
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

Successfully merging this pull request may close these issues.

2 participants