Skip to content

Feature/enhanced my grooves and auto speedup#53

Open
AdarBahar wants to merge 2 commits intomontulli:masterfrom
AdarBahar:feature/enhanced-my-grooves-and-auto-speedup
Open

Feature/enhanced my grooves and auto speedup#53
AdarBahar wants to merge 2 commits intomontulli:masterfrom
AdarBahar:feature/enhanced-my-grooves-and-auto-speedup

Conversation

@AdarBahar
Copy link
Copy Markdown

No description provided.

…s support

Major enhancements:
1. Added 'My Grooves' modal to save grooves you've created (edit, delete and save)
2. Added search capability to 'My Grooves' with real-time filtering and highlighting
3. Added 'Set default' to Auto Speed Up settings for quick access
4. Added new green Play+ button to automatically play with auto speed up enabled and default values
5. Enhanced UI consistency and user experience across all components
6. Included the 2 new Toms branch for expanded drum kit support

Technical improvements:
- Widened My Grooves modal (150px → 450px) for better readability
- Implemented edit functionality with visual feedback (SAVE → UPDATE)
- Real-time search with term highlighting and keyboard support
- Independent play button states with separate auto speed up control
- Enhanced error handling and cross-browser compatibility
- Added comprehensive documentation and testing utilities

Files modified:
- index.html: Auto speed up default checkbox
- css/groove_writer_orange.css: Modal styling and UI enhancements
- css/groove_display_orange.css: Green Play+ button styling
- js/groove_writer.js: Core functionality for edit, search, and auto speed up
- js/groove_utils.js: MIDI player enhancements and button management
- js/groove_display.js: Tom2 drum support addition
…-90%+. Fixed corrupted soundfont data, added HTML5 audio system with MP3 samples, and fixed deploy script to include drum samples. All drum sounds now working properly.
@AdarBahar AdarBahar closed this Jun 15, 2025
@AdarBahar AdarBahar reopened this Jun 15, 2025
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.

1 participant