downlords-faf-client icon indicating copy to clipboard operation
downlords-faf-client copied to clipboard

[Feature]: Mark Watched Replays with Visual Indicators

Open magge-faf opened this issue 1 year ago • 5 comments

Is your feature request related to a problem?

Losing track of which replays have already been watched.

Describe the solution you'd like

It would be a nice QOL improvement if watched replays were marked with an icon, different shade of color, watched-date/time or an outline to indicate they have already been viewed.

E.g.:

image

Describe alternatives you've considered

A simple checkmark-icon or watched-tag

magge-faf avatar Aug 28 '24 06:08 magge-faf

Should this be saved locally (per installation) or server side?

Brutus5000 avatar Aug 28 '24 08:08 Brutus5000

For easier and faster implementation, I suggest doing it locally for now. If the QOL feature gains more appeal, it can always be refactored to work with the server.

magge-faf avatar Aug 28 '24 10:08 magge-faf

@Sheikah45 gonna take look at this one, but an question here.

I'm planning to use something like - Set<Replay_id, {replay_id, list watchedDates}> , though question is - was there something similarly implemented -> e.g. saved locally and read afterwards. Some sort of service? (I don't it belong to prefs) Is there a service that I could follow for this feature? Any tips(keyword are enough)? Otherwise I thinking about inventing a wheel again 🚀

sulikdan avatar Jan 06 '25 14:01 sulikdan

I actually do think the best place is probably in the prefs maybe or I guess it could be some other json persistent file that just contains the dates and replay IDs.

There isn't currently any service that stores purely client side persistent state like this besides the preferences.

Sheikah45 avatar Jan 06 '25 14:01 Sheikah45

Where it would go is probably just in the replay service or replay runner

Sheikah45 avatar Jan 06 '25 14:01 Sheikah45