FreeEMS Issues - Loader
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0000514LoaderUser Interfacepublic2012-01-26 14:352013-11-27 22:08
ReporterFred 
Assigned Tomalcom2073 
PriorityhighSeveritymajorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformLinuxOSDebianOS VersionSid/Unstable
Product Version0.1.0-SNAPSHOT 
Target Version0.1.0Fixed in Version 
Issue TypeBug
Risk of Breakagevery low
Summary0000514: Code Responds To Wrong Button Events
DescriptionInstead of performing an action on button release, if and only if the mouse is still within the button, you perform the action when the mouse goes down, consequently the valid behaviour of holding a button and moving the mouse before release results in an action that it should not. This is bad and wrong. Please fix.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files

Notes
(0001154)
sean94z   
2012-01-26 16:39   
Right, IIRC some buttons behave as they should. Must have been those first couple buttons that were added that slipped though the cracks.
(0002691)
sean94z   
2013-07-07 06:53   
Mike, can you advice as to which signal would be the best solve this problem?
(0002739)
malcom2073   
2013-11-27 21:10   
clicked() is proper, not mousePressed. This handles the case where you click down, move off and release, and does not trigger in that situation.
(0002740)
sean94z   
2013-11-27 22:08   
commit 7637a6f4fe775f60c55c3cd2d2202bcc2739952e
Author: Sean Keys <skeys@athenawave.com>
Date: Wed Nov 27 15:10:13 2013 -0700

    [514] Fixed as per Mikes suggestion. I'm going to assume
    that's as good as it gets within reason.