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

Modify Table list selection can be 'fooled' by dragging #261

Closed
EArmour opened this issue Apr 15, 2015 · 5 comments
Closed

Modify Table list selection can be 'fooled' by dragging #261

EArmour opened this issue Apr 15, 2015 · 5 comments
Labels
bug Confirmed bugs or reports that are very likely to be bugs.

Comments

@EArmour
Copy link

EArmour commented Apr 15, 2015

This is occurring in the latest nightly Windows build.

When modifying the order of fields, the 'move up/down' buttons are disabled when at the top/bottom of the list. But if you click on another row, then hold down the mouse button and drag to one of the end rows, the movement buttons are not properly disabled. So you can click to move the top row up and the bottom row down, both of which crash the application.

The inverse also works, so if you click the bottom row, then drag up one row, the "move down" button will be disabled even though you should be able to move the field. So basically it seems like the listener for disabling those buttons triggers on the mouse click, not the actual final selection after dragging around.

2015-04-15 15_23_01-db browser for sqlite - d__dev__pycharm projects_smallthings_static_mwd db

@justinclift justinclift added the bug Confirmed bugs or reports that are very likely to be bugs. label Apr 15, 2015
@justinclift
Copy link
Member

Thanks, that definitely sounds like a bug. 😉

@MKleusberg
Copy link
Member

Haha nice find 👍 Thanks for reporting!
This should be fixed in the next nightly build.

@justinclift justinclift added this to the 3.6 milestone Apr 17, 2015
@justinclift
Copy link
Member

@EArmour Would you be ok to try out the next nightly build (tomorrow), and let us know if it's working like it should now? 😄

@EArmour
Copy link
Author

EArmour commented Apr 18, 2015

Alright, looks good now! Thanks for the quick response 👍

@justinclift
Copy link
Member

Awesome! Thanks for letting us know @EArmour. 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed bugs or reports that are very likely to be bugs.
Projects
None yet
Development

No branches or pull requests

3 participants