Comment 4 for bug 1088031

Revision history for this message
Max Linke (max-linke) wrote :

Hi Callum

I just looked over your changes. Here are some comments

- include 'lock BPM' again
- if the BPM is locked the all options to change the BPM should be grayed out (see m_pClearBeatsAction)
- I think all this BPM stuff could go into a submenu like playlists. We have a lot of option there right now and this
  could get cramped on small screens
- you've written in a TODO that the changing of the BPM should be done in a seperate thread. Did you notice a long
  GUI-freeze when you worked on a large selection of tracks?