Sigma-Web-Dev-Course icon indicating copy to clipboard operation
Sigma-Web-Dev-Course copied to clipboard

Update script.js, Added Features to Enhance UX

Open TalhaUsmanOfficial opened this issue 10 months ago • 3 comments

✨ Added keyboard controls and enhanced mute functionality

✅ Added spacebar event listener to play/pause the song
✅ Enhanced the mute button: volume slider now updates (0 when muted, 100 when unmuted)
✅ Implemented seek controls: Left Arrow rewinds 5s, Right Arrow forwards 5s ✅ Enhanced Volume Control: If the song is muted and user increases or decreases the volume then the volume will automatically unmuted.

TalhaUsmanOfficial avatar Mar 08 '25 19:03 TalhaUsmanOfficial

@TalhaUsmanOfficial you can add input me search feature like when you are in one of the playlist then if you search the name of the particular song name you can compare it with the song inside the playlist or just make the array of song names and use for and ifelse to check and then do hidden to other song cards and only display that song

On Sun, 9 Mar, 2025, 11:36 am MUHAMMAD TALHA USMAN, < @.***> wrote:

@.**** commented on this pull request.

Hi @ZarhanMemon https://github.com/ZarhanMemon, thanks for reviewing! Could you please clarify what’s still in progress or any changes you’d like me to make?

— Reply to this email directly, view it on GitHub https://github.com/CodeWithHarry/Sigma-Web-Dev-Course/pull/255#pullrequestreview-2669324124, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKBFVCGCMVBKMNDKHFFBLZL2TPLABAVCNFSM6AAAAABYTKML4GVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDMNRZGMZDIMJSGQ . You are receiving this because you were mentioned.Message ID: @.***>

ZarhanMemon avatar Mar 09 '25 06:03 ZarhanMemon

Got it, @ZarhanMemon! I'll add a search feature to filter songs within the playlist by name. Thanks for the suggestion! 🚀

@TalhaUsmanOfficial you can add input me search feature like when you are in one of the playlist then if you search the name of the particular song name you can compare it with the song inside the playlist or just make the array of song names and use for and ifelse to check and then do hidden to other song cards and only display that song On Sun, 9 Mar, 2025, 11:36 am MUHAMMAD TALHA USMAN, < @.> wrote: @.* commented on this pull request. Hi @ZarhanMemon https://github.com/ZarhanMemon, thanks for reviewing! Could you please clarify what’s still in progress or any changes you’d like me to make? — Reply to this email directly, view it on GitHub <#255 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKBFVCGCMVBKMNDKHFFBLZL2TPLABAVCNFSM6AAAAABYTKML4GVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDMNRZGMZDIMJSGQ . You are receiving this because you were mentioned.Message ID: @.***>

TalhaUsmanOfficial avatar Mar 09 '25 09:03 TalhaUsmanOfficial

@ZarhanMemon I've added the search functionality as requested! Now, users can search for songs within a playlist, and only matching songs will be displayed. Let me know if any further improvements are needed. 😊

@TalhaUsmanOfficial you can add input me search feature like when you are in one of the playlist then if you search the name of the particular song name you can compare it with the song inside the playlist or just make the array of song names and use for and ifelse to check and then do hidden to other song cards and only display that song On Sun, 9 Mar, 2025, 11:36 am MUHAMMAD TALHA USMAN, < @.> wrote: @.* commented on this pull request. Hi @ZarhanMemon https://github.com/ZarhanMemon, thanks for reviewing! Could you please clarify what’s still in progress or any changes you’d like me to make? — Reply to this email directly, view it on GitHub <#255 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKBFVCGCMVBKMNDKHFFBLZL2TPLABAVCNFSM6AAAAABYTKML4GVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDMNRZGMZDIMJSGQ . You are receiving this because you were mentioned.Message ID: @.***>

TalhaUsmanOfficial avatar Mar 09 '25 10:03 TalhaUsmanOfficial