Comment 7 for bug 1088031
Revision history for this message
| Max Linke (max-linke) wrote Re: [Bug 1088031] Re: Add Double BPM and Halve BPM to contextual menu | #7 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
Hcan you please create the patch in your mixxx directory with
bzr diff > bpm.patch
so that I can directly apply it. With your current version I need to
edit a lot by hand so that I can apply it to trunk.
On Mon, 13 May 2013 17:36:07 -0000
Callum Styan <email address hidden> wrote:
> Here's an updated version of the patch.
>
> I added the new BPM actions and menu to the deconstructor, and added
> all the BPM related actions into one BPM submenu. Finally, I made it
> so that if the BPM is locked then the Double, Halve, and 2/3 BPM
> items are greyed out.
>
> As I mentioned above, I added the TODO comment which was present in
> most of the other 'slot' functions which acted on multiple selected
> items. I added the comment for consistency's sake, not because I
> experienced any UI lock issues. Perhaps the threading of these
> should be added as another wishlist item?
>
> ** Attachment added: "updated diff."
> https:/
>