Skip to content

Feat:social lyric sharing#695

Open
Sai-Adithya-M wants to merge 1 commit intoAOSSIE-Org:devfrom
Sai-Adithya-M:feat/social-lyric-sharing
Open

Feat:social lyric sharing#695
Sai-Adithya-M wants to merge 1 commit intoAOSSIE-Org:devfrom
Sai-Adithya-M:feat/social-lyric-sharing

Conversation

@Sai-Adithya-M
Copy link

@Sai-Adithya-M Sai-Adithya-M commented Jan 8, 2026

Description

This PR introduces the Social Lyric Sharing feature, allowing users to share their favorite lyrics from Resonate directly to social media as formatted image cards.

Context: Previously, the app only supported passive lyric reading. This change encourages social engagement by allowing users to share content.

Summary of Changes:

  • New Widget: LyricCard - A widget that composites the story cover, title, and current lyric line into a visually appealing card.
  • Controller Logic: Added shareCurrentLyric in ChapterPlayerController to capture the LyricCard
    as a PNG using RepaintBoundary and share it using share_plus.
  • UI: Added a "Share Lyric" button to ChapterPlayScreen.

Dependencies:

  • share_plus
  • path_provider

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

I have tested this feature using both widget tests and static analysis.

Tests:

  1. Widget Test: lyric_card_test.dart
  • Verifies that the LyricCard widget renders correctly with the provided data (text, image, icons).
  • Command: flutter test test/views/widgets/lyric_card_test.dart

Configuration:

  • Flutter SDK: Stable
  • Platform: iOS/Android (via code logic), mocked in tests.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings

Closes #697

Maintainer Checklist

@coderabbitai
Copy link

coderabbitai bot commented Jan 8, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

🎉 Welcome @Sai-Adithya-M!
Thank you for your pull request! Our team will review it soon. 🔍

  • Please ensure your PR follows the contribution guidelines. ✅
  • All automated tests should pass before merging. 🔄
  • If this PR fixes an issue, link it in the description. 🔗

We appreciate your contribution! 🚀

@Sai-Adithya-M Sai-Adithya-M changed the title Feat/social lyric sharing Feat:social lyric sharing Jan 8, 2026
@M4dhav
Copy link
Contributor

M4dhav commented Jan 10, 2026

Please open an issue for this, remove the unrelated commits from your PR, fix merge conflicts

@Sai-Adithya-M Sai-Adithya-M force-pushed the feat/social-lyric-sharing branch from 89983df to edd9592 Compare January 10, 2026 13:29
@Sai-Adithya-M Sai-Adithya-M requested a review from M4dhav as a code owner January 10, 2026 13:29
@Sai-Adithya-M Sai-Adithya-M force-pushed the feat/social-lyric-sharing branch from edd9592 to 89983df Compare January 10, 2026 13:34
@M4dhav M4dhav linked an issue Jan 10, 2026 that may be closed by this pull request
@M4dhav M4dhav added the enhancement New feature or request label Jan 10, 2026
@M4dhav
Copy link
Contributor

M4dhav commented Jan 10, 2026

The PR still has some unrelated commits, see here - https://github.com/AOSSIE-Org/Resonate/pull/695/commits

@Sai-Adithya-M Sai-Adithya-M force-pushed the feat/social-lyric-sharing branch from 42e3ea4 to 9ccb5a6 Compare January 10, 2026 21:04
@M4dhav
Copy link
Contributor

M4dhav commented Jan 11, 2026

Can you share a video of the functionality please

@Sai-Adithya-M
Copy link
Author

Can you share a video of the functionality please

Screen.Recording.2026-01-12.at.12.29.26.AM.mov

@M4dhav
Copy link
Contributor

M4dhav commented Jan 14, 2026

Looks like you are running it on the macOS platform. Could you please verify it runs on iOS and Android? This is necessary to do separately as some functionalities do not run on all platforms.

Also, please share a video of it on the mobile platforms

@Sai-Adithya-M Sai-Adithya-M force-pushed the feat/social-lyric-sharing branch from 2dade92 to 82768b0 Compare January 14, 2026 17:06
@Sai-Adithya-M
Copy link
Author

Looks like you are running it on the macOS platform. Could you please verify it runs on iOS and Android? This is necessary to do separately as some functionalities do not run on all platforms.

Also, please share a video of it on the mobile platforms

image image

@Sai-Adithya-M
Copy link
Author

Looks like you are running it on the macOS platform. Could you please verify it runs on iOS and Android? This is necessary to do separately as some functionalities do not run on all platforms.

Also, please share a video of it on the mobile platforms

WhatsApp.Video.2026-01-19.at.2.27.14.AM.mp4

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feat: Social Lyric Sharing

2 participants