qwt / Bugs / #163 QwtPickerPolygonMachine: wrong key/command mapping

#163 QwtPickerPolygonMachine: wrong key/command mapping

Status: closed

Owner: Uwe Rathmann

Labels: None

Priority: 5

Updated: 2012-07-20

Created: 2012-05-28

Creator: srunkel

Private: No

The documentation states:
"
Pressing QwtEventPattern::MouseSelect1 or QwtEventPattern::KeySelect1 starts the selection and selects the first point, or appends a point. Pressing QwtEventPattern::MouseSelect2 or QwtEventPattern::KeySelect2 appends the last point and terminates the selection
"

However, that intuitive mapping is not implemented.

Please find attached a patch against 6.01 to achive that.

Discussion

srunkel - 2012-05-28

qwt_picker_machine.cpp.patch

qwt_picker_machine.cpp.patch

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

srunkel - 2012-06-10

assigned_to: nobody --> rathmann

status: open --> pending-works-for-me
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Uwe Rathmann - 2012-06-11

Indeed the implementation differs from the documentation and I agree, that the documented model is more intuitive - but changing the code will break all applications using this state machine.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Uwe Rathmann - 2012-06-11

status: pending-works-for-me --> open-accepted
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Uwe Rathmann - 2012-07-20

Fixed in 6.0 and trunk.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Uwe Rathmann - 2012-07-20

status: open-accepted --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Anonymous