Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Added Quick Lyrics Search Activity #108

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

BobbyESP
Copy link
Collaborator

Added a new activity for quick lyrics search that can be launched from an intent. The activity fetches lyrics for a given song and artist using the LyricsProviderService. The user can then send the lyrics back to the calling app.

This feature is useful for quickly sharing lyrics without having to open the main app.

BobbyESP and others added 11 commits November 14, 2024 16:34
Added a new activity for quick lyrics search that can be launched from an intent.
The activity fetches lyrics for a given song and artist using the LyricsProviderService.
The user can then send the lyrics back to the calling app.

This feature is useful for quickly sharing lyrics without having to open the main app.
Removed custom window insets handling code and system window insets disabling. This simplifies the activity's layout and removes unnecessary code for managing window insets.
Redesigned the Quick Search page UI with new components and structure, enhancing user experience.
Added quick access buttons for sending lyrics and navigating to settings.
Improved lyrics display with a scrollable and expandable section.
Integrated album art using Coil library for improved visual appeal.
Added animations for smooth transitions.
Updated string resources for better localization.

Signed-off-by: Gabriel Fontán <gabilessto@gmail.com>
- Improved code style and formatting.

Signed-off-by: Gabriel Fontán <gabilessto@gmail.com>
- Added monospace font for timestamps in lyrics to improve readability.
- Changed timestamp color to better contrast with the background.
- Added song name validation to prevent crashes from invalid intent data.
- Improved the UI of the QuickLyricsSearchPage for better clarity and accessibility.
- Added content descriptions to icons for improved accessibility.

Signed-off-by: Gabriel Fontán <gabilessto@gmail.com>
- Implemented Material Shared Axis transitions for screen changes, providing a visually appealing and intuitive flow.
- Introduced fade-through animations for specific elements, ensuring seamless transitions between states.
- Added slide-in animations for vertically arranged content, improving visibility and engagement.
- Adjusted animation durations and easing functions for a more polished and natural feel.
- Created reusable animation components for better maintainability and consistency across the app.

Signed-off-by: Gabriel Fontán <gabilessto@gmail.com>
Signed-off-by: Gabriel Fontán <gabilessto@gmail.com>
Moved the initialization of UserSettingsController to the onCreate method of QuickLyricsSearchActivity due to problems with use of Context before initialization.

Signed-off-by: Gabriel Fontán <gabilessto@gmail.com>
- Introduced shared element transitions for smoother navigation between screens.
- Added an error card to display error messages with stack traces.
- Updated the UI to use the new error card for error handling.
- Updated the language level to JDK 21 and the project JDK to jbr-21.
- Added a string resource for unknown errors.
- Refactored animation specs for better performance and visual appeal.
- Improved the overall user experience by enhancing screen transitions and providing more informative error messages.

Signed-off-by: Gabriel Fontán <gabilessto@gmail.com>
@BobbyESP BobbyESP marked this pull request as ready for review November 20, 2024 14:45
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