Comment 5 for bug 1088031

Revision history for this message
Callum Styan (callumstyan) wrote :

Hi Max

I'll be submitting another patch today with some changes but here are answers for a few of your comments:

- lock BPM wasn't removed, lock/unlock BPM and the 3 new settings are now all in a submenu of the context menu
- you're right I should grey the options out if the BPM is locked
- the other slot functions contain that TODO comment, I left it in for consistency's sake because there is the possibility that selecting a large number of tracks could lock the UI, but at the moment none of the slot functions make use of a seperate thread maybe that should be another wishlist item?