Skip to content

Feature // Load next song MIDI command - #2810

Merged
m-m-adams merged 29 commits into
SynthstromAudible:communityfrom
soymonitus:monitus/next_song_command
Nov 21, 2024
Merged

Feature // Load next song MIDI command#2810
m-m-adams merged 29 commits into
SynthstromAudible:communityfrom
soymonitus:monitus/next_song_command

Conversation

@soymonitus

@soymonitus soymonitus commented Oct 15, 2024

Copy link
Copy Markdown
Collaborator

This adds a new learnable midi command (in button LOAD) that will autoload next song in the current folder

  • Fix issue where the loaded song still displays the info of the previous song
  • Fix crash when receiving the midi command while song is in the process of being loaded
  • Make it ignore commands while processing another auto-next command
  • Make it skip folders
  • Add docs
  • Refactor LoadUI so it loading code is decoupled from UI code

@github-actions

github-actions Bot commented Oct 15, 2024

Copy link
Copy Markdown
Contributor

Test Results

103 tests  ±0   103 ✅ ±0   1s ⏱️ ±0s
 15 suites ±0     0 💤 ±0 
 15 files   ±0     0 ❌ ±0 

Results for commit 4c83f54. ± Comparison against base commit 3206adb.

♻️ This comment has been updated with latest results.

@seangoodvibes
seangoodvibes marked this pull request as draft October 15, 2024 16:02
@soymonitus soymonitus changed the title [Draft] Load next song MIDI command Feature // Load next song MIDI command Oct 16, 2024
@soymonitus
soymonitus marked this pull request as ready for review October 16, 2024 20:40
@soymonitus

Copy link
Copy Markdown
Collaborator Author

Ready for review!

Comment thread src/deluge/gui/ui/load/load_song_ui.cpp Outdated
@SynthstromAudible SynthstromAudible deleted a comment from soymonitus Oct 23, 2024
@seangoodvibes

Copy link
Copy Markdown
Collaborator

@m-m-adams this implementation looks solid but would appreciate a second lens on it. nothing stands out for me.

Comment thread src/deluge/gui/ui/load/load_song_ui.cpp Outdated
Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
Comment thread src/deluge/gui/ui/load/load_song_ui.cpp Outdated
Comment thread src/deluge/gui/views/view.cpp Outdated
@soymonitus

Copy link
Copy Markdown
Collaborator Author

Ok i removed the possibility to learn the command from holding LOAD so it is now only accessible from settings menu. @m-m-adams

@seangoodvibes seangoodvibes added this to the Release 1.3 milestone Nov 4, 2024
# Conflicts:
#	docs/community_features.md
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
seangoodvibes added a commit to seangoodvibes/DelugeFirmware that referenced this pull request Jan 15, 2026
* Load next song midi command

* fix

* Fixed!

* Fix ui

* dbt format

* Skip animations while executing load next song command

* Skip folders and load next song only if it is a file. Exit browser if song not found

* format

* Docs

* Update menu hierarchies

* menu hierarchies 7seg

* Actualizar load_song_ui.cpp

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>

* Remove ability to learn NEXT SONG from LOAD button directly. Only allowed now from SETTINGS menu

* Make it independent from UI

* Fixed crash

* Simplifying stuff

* Apply better safety checks

* Fix compile issue

* refactor

* Format

* Better check as it allows more flexibility

* This provides more stable checks

---------

Co-authored-by: Sean Ditny <138174805+seangoodvibes@users.noreply.github.com>
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.

3 participants