Skip to content

Automation View ~ Bug fixes and added select scrolling of patch cables#1607

Merged
seangoodvibes merged 3 commits into
SynthstromAudible:communityfrom
seangoodvibes:Select-Scroll-Patch-Cables-in-Automation-View
Apr 1, 2024
Merged

Automation View ~ Bug fixes and added select scrolling of patch cables#1607
seangoodvibes merged 3 commits into
SynthstromAudible:communityfrom
seangoodvibes:Select-Scroll-Patch-Cables-in-Automation-View

Conversation

@seangoodvibes

@seangoodvibes seangoodvibes commented Mar 27, 2024

Copy link
Copy Markdown
Collaborator
  • Added select scrolling of patch cables that exist (they are dynamically added to the end of the non-patch cable param scrolling list for synths and kit rows)
  • Handled scenario where you selected a patch cable previously and it no longer exists when re-entering automation view. In this case it now sends you back to the automation overview.
  • Fixed select scrolling bug that would cause interpolation shortcut to blink profusely
  • Updated patch cable name displayed to show both sources if it's a patch cable with two sources

@seangoodvibes seangoodvibes added the cherry-pick Commit to cherry pick to release branch label Mar 27, 2024
@seangoodvibes seangoodvibes added this to the Release 1.1 milestone Mar 27, 2024
@seangoodvibes seangoodvibes marked this pull request as draft March 27, 2024 21:01
@github-actions

github-actions Bot commented Mar 27, 2024

Copy link
Copy Markdown
Contributor

Test Results

9 tests  ±0   9 ✅ ±0   0s ⏱️ ±0s
2 suites ±0   0 💤 ±0 
2 files   ±0   0 ❌ ±0 

Results for commit e365df4. ± Comparison against base commit 989871e.

♻️ This comment has been updated with latest results.

@seangoodvibes seangoodvibes force-pushed the Select-Scroll-Patch-Cables-in-Automation-View branch from f8ff4ed to 1ec5e09 Compare March 27, 2024 21:08
@seangoodvibes seangoodvibes marked this pull request as ready for review March 27, 2024 21:09
@seangoodvibes seangoodvibes changed the title Select Scroll Patch Cables and Render Name for Patch Cable with Two Sources Automation View ~ Select Scroll Patch Cables and Render Name for Patch Cable with Two Sources Mar 27, 2024
@seangoodvibes seangoodvibes force-pushed the Select-Scroll-Patch-Cables-in-Automation-View branch from 1ec5e09 to 87ad6a7 Compare March 27, 2024 21:12
@seangoodvibes seangoodvibes changed the title Automation View ~ Select Scroll Patch Cables and Render Name for Patch Cable with Two Sources Automation View ~ Bug fixes and added select scrolling of patch cables Mar 27, 2024
@seangoodvibes seangoodvibes force-pushed the Select-Scroll-Patch-Cables-in-Automation-View branch from 3ec214b to 211c096 Compare March 27, 2024 21:18
…ources

- Added select scrolling of patch cables that exist (they are dynamically added to the end of the non-patch cable param scrolling list for synths and kit rows)
- Handled scenario where you selected a patch cable previously and it no longer exists when re-entering automation view. In this case it now sends you back to the automation overview.
- Fixed select scrolling bug that would cause interpolation shortcut to blink profusely
- Updated patch cable name displayed to show both sources if it's a patch cable with two sources
@seangoodvibes seangoodvibes force-pushed the Select-Scroll-Patch-Cables-in-Automation-View branch from 211c096 to f82e010 Compare March 27, 2024 21:22
Comment thread src/deluge/gui/views/automation_view.cpp
@seangoodvibes seangoodvibes added this pull request to the merge queue Apr 1, 2024
Merged via the queue into SynthstromAudible:community with commit f8f428a Apr 1, 2024
github-actions Bot pushed a commit that referenced this pull request Apr 1, 2024
#1607)

* Select Scroll Patch Cables and Render Name for Patch Cable with Two Sources

- Added select scrolling of patch cables that exist (they are dynamically added to the end of the non-patch cable param scrolling list for synths and kit rows)
- Handled scenario where you selected a patch cable previously and it no longer exists when re-entering automation view. In this case it now sends you back to the automation overview.
- Fixed select scrolling bug that would cause interpolation shortcut to blink profusely
- Updated patch cable name displayed to show both sources if it's a patch cable with two sources

* Update code and comments for clarity

* Update documentation
sapphire-arches pushed a commit that referenced this pull request Apr 2, 2024
#1607)

* Select Scroll Patch Cables and Render Name for Patch Cable with Two Sources

- Added select scrolling of patch cables that exist (they are dynamically added to the end of the non-patch cable param scrolling list for synths and kit rows)
- Handled scenario where you selected a patch cable previously and it no longer exists when re-entering automation view. In this case it now sends you back to the automation overview.
- Fixed select scrolling bug that would cause interpolation shortcut to blink profusely
- Updated patch cable name displayed to show both sources if it's a patch cable with two sources

* Update code and comments for clarity

* Update documentation
tastycode pushed a commit to tastycode/DelugeFirmware that referenced this pull request Jul 3, 2024
SynthstromAudible#1607)

* Select Scroll Patch Cables and Render Name for Patch Cable with Two Sources

- Added select scrolling of patch cables that exist (they are dynamically added to the end of the non-patch cable param scrolling list for synths and kit rows)
- Handled scenario where you selected a patch cable previously and it no longer exists when re-entering automation view. In this case it now sends you back to the automation overview.
- Fixed select scrolling bug that would cause interpolation shortcut to blink profusely
- Updated patch cable name displayed to show both sources if it's a patch cable with two sources

* Update code and comments for clarity

* Update documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick Commit to cherry pick to release branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants