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

silx.gui: Fixed support of PySide6.4 enums #3737

Merged
merged 1 commit into from
Dec 15, 2022

Conversation

t20100
Copy link
Member

@t20100 t20100 commented Dec 15, 2022

This PR solves issue #3730 for QFileDialog.

I also checked similar cases for all QDialog and inherited classes.

closes #3730 even if there is probably other enum issues but the one reported should be fixed.

@t20100 t20100 added this to the Next release milestone Dec 15, 2022
@t20100 t20100 requested a review from vasole December 15, 2022 14:27
@t20100
Copy link
Member Author

t20100 commented Dec 15, 2022

BTW, I'm using qt.QFileDialog.DontUseNativeDialog rather than qt.QFileDialog.Option.DontUseNativeDialog to keep supporting PyQt5 < 5.11

Copy link
Member

@vasole vasole left a comment

Choose a reason for hiding this comment

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

Enough for the time being.

@vasole
Copy link
Member

vasole commented Dec 15, 2022

I wonder if we could check in our qt.py module for the presence of the appropriate enums among the attributes of qt.QFileDialog in order to get a more generic solution.

As you already point out, I am sure there are also issues with the modes.

@vasole vasole merged commit 3ffaa97 into silx-kit:master Dec 15, 2022
@t20100 t20100 deleted the fix-pyside6.4-dialog branch December 16, 2022 08:15
@t20100 t20100 mentioned this pull request Dec 16, 2022
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.

[GUI] PySide6 6.4 issues with file dialog enums
2 participants