Skip to content

Fix some bugs with LFO waveform drag&drop - #4227

Merged
PhysSong merged 2 commits into
LMMS:stable-1.2from
PhysSong:lfodisplay
Mar 14, 2018
Merged

Fix some bugs with LFO waveform drag&drop#4227
PhysSong merged 2 commits into
LMMS:stable-1.2from
PhysSong:lfodisplay

Conversation

@PhysSong

@PhysSong PhysSong commented Mar 8, 2018

Copy link
Copy Markdown
Member

Update LFO waveform display after dropping a sample file. Fixes #3307.
This PR also fixes a broken functionality, "drag&drop from sample tracks".

* Update LFO waveform display after dropping a sample file
* Fix broken drag&drop from sample tracks
@zonkmachine

Copy link
Copy Markdown
Contributor

This PR also fixes a broken functionality, "drag&drop from sample tracks".

Nice catch!

@PhysSong

PhysSong commented Mar 9, 2018

Copy link
Copy Markdown
Member Author

Maybe out of scope, but is it better to activate user-wave mode automatically when a sample file is dropped?

@zonkmachine

Copy link
Copy Markdown
Contributor

Maybe out of scope, but is it better to activate user-wave mode automatically when a sample file is dropped?

Yes. That would be better.

@PhysSong

Copy link
Copy Markdown
Member Author

Maybe out of scope, but is it better to activate user-wave mode automatically when a sample file is dropped?

Yes. That would be better.

Done via e4f0d25.

@zonkmachine

Copy link
Copy Markdown
Contributor

Done via e4f0d25.

Better.

@PhysSong
PhysSong merged commit b4e7806 into LMMS:stable-1.2 Mar 14, 2018
@PhysSong
PhysSong deleted the lfodisplay branch March 14, 2018 05:48
sdasda7777 pushed a commit to sdasda7777/lmms that referenced this pull request Jun 28, 2022
* Change to user-wave mode and update display after dropping a sample
* Fix broken drag&drop from sample tracks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LFO - Glitch on dropping a user wave/sample

2 participants