libretro / libretro/RetroArch

RFE: icons in playlists with custom names

Open
#15,976 1 comment 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C
Stars
14.1k
Forks
2.2k
Avg merge
7h 35m
Merged PRs (30d)
51

Description

First and foremost consider this:

  • Only RetroArch bugs should be filed here. Not core bugs or game bugs
  • This is not a forum or a help section, this is strictly developer oriented

Description

From thinking about for a bit, I understand the likely reason why playlists depend on filename instead of a internal property to load the main icon, is to be able to load icons before parsing playlists when entering the playlist tab so there are no delays.

But I'd still like to have custom names and a icon. Therefore I'd like to propose a rfe to ignore text added after a filename match when starting from 0.

For instance, you could name a playlist the custom names:
Sony - Sony PlayStation (Japan)
Sony - Sony PlayStation (Hacks)
Scummvm (AGS)

And to match the icon only the bolded part would be considered, while still allowing playlist theme divisions with icons.

Expected behavior

At first I expected this could be done more systematically and without forcing (part) of a filename by modifying the playlist JSON format, but finally understood the filename requirement was likely about the playlist tab responsiveness and not being forced to parse playlists before displaying icons, so came to this alternative idea.

It has the advantage of keeping same system icons near each other on the sort order

Actual behavior

Custom playlist names have the default icon, with absolutely no way to avoid it if you want custom theme playlists or want to divide subsystems.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No source file or test is named. Start by tracing the playlist tab's icon matching and playlist-name handling in the C frontend, then determine how matching can ignore the proposed suffix without adding playlist parsing delays. Done means custom-named playlists use the matching base icon while retaining responsive playlist display.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
frontend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.