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

Qt6 migration - pt2 #292

Merged
merged 16 commits into from
May 16, 2022
Merged

Qt6 migration - pt2 #292

merged 16 commits into from
May 16, 2022

Conversation

ParkerK
Copy link
Contributor

@ParkerK ParkerK commented May 13, 2022

A few more changes to make this qt6 compatible, this fixes the 'Monitor' tab for Cura 5.0


For #287

@ParkerK
Copy link
Contributor Author

ParkerK commented May 13, 2022

@Jeredian #288 (comment)

@Elkin-Vasily , Looks like we need to change also min Cura version since this patch, because we have breaking changes for Cura 4.* , don't we?

I thought I saw another plugin that had somehow managed to put the legacy (<5.0) code in a new directory for backward compatibility, I will see if I can find it

@@ -61,7 +61,7 @@ Item {
onClicked: Cura.MachineManager.printerOutputDevices[0].requestAuthentication();
// style: UM.Theme.styles.print_setup_action_button;
text: catalog.i18nc("@action:button", "Request Access");
tooltip: catalog.i18nc("@info:tooltip", "Send access request to the printer");
// tooltip: catalog.i18nc("@info:tooltip", "Send access request to the printer");
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we keep somehow tooltip usage, or it's deprecated in new release?

Copy link
Contributor Author

@ParkerK ParkerK May 13, 2022

Choose a reason for hiding this comment

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

This is something I'm still trying to figure out. I know that attempting to use the tooltip property for Qt's Button will cause an error, as it's been removed. It might still be in Cura.PrimaryButton or another ButtonType? If not, I think it can be added via a onHover method or something. I've just commented them out for now so the plugin will load in Cura 5.0, but it certainly should be addressed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Found it. https://doc-snapshots.qt.io/qt6-dev/qml-qtquick-controls2-tooltip.html#details

I will update all tooltips accordingly :)

@ParkerK
Copy link
Contributor Author

ParkerK commented May 13, 2022

Found it https://github.com/emtrax-ltd/Cura2MoonrakerPlugin seems to be able to handle Qt5 and 6 in the same package. I didn't look too much into it yet though.

Same with https://github.com/fieldOfView/Cura-OctoPrintPlugin

Also, I just wanted to thank you for letting me help out with this, I use the plugin daily, so I'm just happy I'm able to give back :)

@sonarcloud
Copy link

sonarcloud bot commented May 13, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@Jeredian Jeredian added the pull request: community Improvements or additions from the community label May 16, 2022
@Jeredian Jeredian merged commit 22f2b24 into PrintMakerLab:develop May 16, 2022
@Jeredian Jeredian mentioned this pull request May 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pull request: community Improvements or additions from the community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants