cancelEvent event slot does not exist #28

Closed
opened 3 years ago by duke · 3 comments
duke commented 3 years ago
Owner

Line 292 of this file uses an event slot cancelEvent which doesn't exist and generates a warning in STDERR but does not have any visible effect in the GUI. Except that the FirstTimeWizard::cancelEvent function will never run, which means users are not asked to confirm about quitting setup.

This could be related to the version of QT5 in use and is not clear if it ever worked correctly.

Line 292 of this file uses an event slot `cancelEvent` which doesn't exist and generates a warning in STDERR but does not have any visible effect in the GUI. Except that the FirstTimeWizard::cancelEvent function will never run, which means users are not asked to confirm about quitting setup. This could be related to the version of QT5 in use and is not clear if it ever worked correctly.
Poster
Owner

This happens to me with QT 5.12.8

This happens to me with QT 5.12.8
Collaborator

I figured this one out which should hopefully help with fixing other related issues. The problem is cancelEvent is defined as a public function instead of a slot in header: https://git.hush.is/hush/SilentDragonLite/src/branch/dev/src/firsttimewizard.h#L26

Moving to slot definitions just below and the cancelEvent gets called and alert shows to confirm quitting setup.

I figured this one out which should hopefully help with fixing other related issues. The problem is cancelEvent is defined as a public function instead of a slot in header: https://git.hush.is/hush/SilentDragonLite/src/branch/dev/src/firsttimewizard.h#L26 Moving to slot definitions just below and the cancelEvent gets called and alert shows to confirm quitting setup.
duke commented 1 year ago
Poster
Owner

seems like this was done a while ago, closing

seems like this was done a while ago, closing
duke closed this issue 1 year ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.