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

fix #278988 Copy and paste notes in Piano Roll Editor #5808

Conversation

blackears
Copy link
Contributor

Can now copy and paste notes in piano roll editor using the right click menu.

mscore/pianoview.cpp Outdated Show resolved Hide resolved
mscore/pianoview.cpp Outdated Show resolved Hide resolved
mscore/pianoview.cpp Outdated Show resolved Hide resolved
@blackears blackears force-pushed the 278988-paste-non-contigious-notes-in-PRE-2 branch 2 times, most recently from e534f12 to b6ab1d8 Compare March 14, 2020 01:21
mscore/pianoview.cpp Outdated Show resolved Hide resolved
@blackears blackears force-pushed the 278988-paste-non-contigious-notes-in-PRE-2 branch from b6ab1d8 to f786de5 Compare March 14, 2020 08:36
@dmitrio95 dmitrio95 added the strings Affects translatable strings label Mar 19, 2020
mscore/pianoview.cpp Outdated Show resolved Hide resolved
mscore/pianoview.cpp Outdated Show resolved Hide resolved
@blackears blackears force-pushed the 278988-paste-non-contigious-notes-in-PRE-2 branch from f786de5 to 08857b7 Compare March 20, 2020 01:14
@blackears blackears force-pushed the 278988-paste-non-contigious-notes-in-PRE-2 branch 2 times, most recently from 46f5c70 to cca23aa Compare April 3, 2020 17:13
@RobFog
Copy link

RobFog commented Apr 17, 2020

Rebase needed.

@blackears blackears force-pushed the 278988-paste-non-contigious-notes-in-PRE-2 branch 3 times, most recently from 1cab9bc to 2926374 Compare April 19, 2020 09:47
@anatoly-os
Copy link
Contributor

@blackears rebase and addressing the review are needed

@blackears blackears force-pushed the 278988-paste-non-contigious-notes-in-PRE-2 branch 2 times, most recently from 1213bb0 to 93b186e Compare April 22, 2020 10:04
@anatoly-os
Copy link
Contributor

@blackears rebase is needed after merging your other PR, sorry :(

@blackears blackears force-pushed the 278988-paste-non-contigious-notes-in-PRE-2 branch 2 times, most recently from ec0bd9c to 8c475f5 Compare April 28, 2020 22:17
@blackears blackears force-pushed the 278988-paste-non-contigious-notes-in-PRE-2 branch from 8c475f5 to 2a93351 Compare April 28, 2020 23:06
@anatoly-os anatoly-os added work in progress not finished work or not addressed review and removed review labels Apr 29, 2020
Can now copy and paste notes in piano roll editor using the right click menu.

Allowing cut and paste of nodes in piano roll editor.


Allowing cut and paste of nodes in piano roll editor.


Cleaning code.

Using Fraction::fromTicks to convert ticks.

Changing variable names.

Updating cut/paste to use Fraction

Adding cut shortcut.

Undo now reverts all notes from paste.

Cleaning up code

Refactoring method

Fixing rebase

Making PRE child of main window.

Can now drag notes in PRE

Fixing layout of PRE levels window

Now showing note blocks when dragging.

Can now drag notes with relative offset

Escape will now cancel drag

Fixing rebase

Removing code that accidentally undid PR musescore#6006

Fixing text, can now set voices of selected notes.
@blackears blackears force-pushed the 278988-paste-non-contigious-notes-in-PRE-2 branch from 2a93351 to be8a618 Compare April 29, 2020 13:56
@anatoly-os anatoly-os removed the work in progress not finished work or not addressed review label May 2, 2020
@anatoly-os anatoly-os merged commit f561851 into musescore:master May 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
strings Affects translatable strings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants