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
PR: Fix merge of 18966 #22202
PR: Fix merge of 18966 #22202
Conversation
This was probably my mistake when rebasing the qt6 support PR 🐑 . |
…202-on-v3.5.x Backport PR #22202 on branch v3.5.x (PR: Fix merge of 18966)
@@ -537,8 +536,7 @@ def fedit(data, title="", comment="", icon=None, parent=None, apply=None): | |||
parent._fedit_dialog.close() | |||
parent._fedit_dialog = dialog | |||
|
|||
if qt_compat._exec(dialog): | |||
return dialog.get() | |||
dialog.show() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change lead to that fedit
does not return any answer (other than None
) leading to that the documentation and probably the examples where the result is printed should be changed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[editted by @tacaswell ]]
Thank you for you interest in Matplotlib, however I have removed the self-portrait as I am not sure why it was posted.
PR Summary
It looks like the merge of #18966 had an issue. #18965 is back. This fixes it.
PR Checklist
Tests and Styling
pytest
passes).flake8-docstrings
and runflake8 --docstring-convention=all
).Documentation
doc/users/next_whats_new/
(follow instructions in README.rst there).doc/api/next_api_changes/
(follow instructions in README.rst there).