tildearrow
tildearrow
Yeah I will do the rest myself. Keep this pull request open so that GitHub detects when I merge it.
> I know but I lost any will to add to the main spaghetti ins edit code in the beginning since SID3 ins gui code promised to be rather big....
> [3ade67a#diff-640c5a5f5f591d9aa98a50f940b0dcafebee17e98ed115938d75377b86d8b175R910](https://github.com/tildearrow/furnace/commit/3ade67a889ecd6816f4893cbec17b6a5cca29f28#diff-640c5a5f5f591d9aa98a50f940b0dcafebee17e98ed115938d75377b86d8b175R910) @tildearrow the sole purpose of the flag was to prevent duty change whenever the new instrument is introduced... So shouldn't it be > > `if(ins->sid3.resetDuty&&chan[c.chan].insChanged)` > > ?...
Clarification needed. You can't drag beyond the edges of a pattern, so this most likely works as intended.
Got it, but: - If you mean selecting the entire pattern and moving it fails, this is intentional. - If you mean selecting an entire row or channel and moving...
Fixing this issue requires new pattern cursor logic. A major change, but one that comes with great benefit.
Please test using latest artifact. It is now possible to select beyond the boundaries of one pattern.
> Adding a new pattern, then using scroll wheel or arrow keys up/down makes the cursor want to warp back to the top of the first pattern, FIxed! > and...
I have improved the undo/redo cursor state logic. Please test using latest artifact.
Regarding second issue - it is finally fixed. Please test with Git master or using latest artifact (when it is done building).