ytmdesktop / ytmdesktop

A Desktop App for YouTube Music

Home Page:https://ytmdesktop.app

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Windows notification shows wrong thumbnail

R00TEAD0 opened this issue · comments

Consent

  • I verified that there is no open/closed issue for the same subject.
  • I understand that YTMDesktop have NO affiliation with Google or YouTube

Current Behavior

Hi there, recently I noticed that notifications on Windows (Windows 11 in my case) show the previous thumbnail instead of the 'playing now' song

image

Expected Behavior

It's a bug so I only want to show it and if possible, contribute to solving it.

Steps To Reproduce

  1. Play a song
  2. Go to the next or previous song
  3. And voila! Maybe...the

YTMDesktop

2.0.0

OS

Windows

OS Version

11

Arch

x64

Installation way

.exe

Anything else?

No response

This is something I've been meaning to diagnose.
During the 4 months of running a test build I never had this happen, but as soon as we went live it happened.

I just saw it today. Because I don't know much about TypeScrypt I'm just speculating according to the syntax. It may be related to how the notifications are shown and the async method that manages the notification process (Maybe some variable that stacks on previous data or something similar after the async function is executed). I will try one of the RC versions to check if the behavior extends to those or not.

This is occurring because we can't obtain the artwork immediately which leads it to being delayed and the state of the song may already be playing before the artwork is received which is causing the notification to show. Unfortunately there's nothing we can do to fix that as YTM doesn't provide it immediately with the rest of the video data. This is part of a broader needed change to help alleviate this. Notifications are probably stuck with needing weird hacky fixes to get around that and they won't be perfect especially if you rely on artwork always existing.