RetroArch icon indicating copy to clipboard operation
RetroArch copied to clipboard

[Menu] Menu sub label finetunes and file names in on-screen notifications

Open davidhedlund opened this issue 1 year ago • 9 comments

Sub-labels

Settings

Replace "file browser" with "File Browser":

  • Settings -> User Interface -> File Browser -> Filter Unknown Extensions: "Filter files being shown in the file browser..."
  • Settings -> User Interface -> File Browser -> Remember Last Used Start Directory: "Open the file browser..."

Replace "system directory" with "System/BIOS directory":

  • Settings -> Core -> System Files are in Content Directory: "Use content directory as system directory."

"Requires restart" sub-label

Asked in https://github.com/libretro/RetroArch/issues/16146#issuecomment-1981823441

"These are all entries in "Settings -> Drivers". Can someone please answer which one that should have the sub-label "Requires restart.", because I don't know how to test them?:

  • Menu: Has "Requires restart."
  • Video: Yes/no?
  • Audio: Yes/no?
  • Microphone: Yes/no?
  • Input: Yes/no?
  • Controller: Yes -- Confirmed by me
  • Record: Yes/no?
  • MIDI: Yes/no?
  • Camera: Yes/no?
  • Location: Yes/no?"

When the "Controller" entry has "Requires restart", document that we encourage users to generate and upload autoconfig files for all controller drivers at https://github.com/libretro/docs/pull/910

Quick Menu

The requirement is not added, it is: "Requires first that file(s) has been saved via option(s) at 'Controls -> Manage Remap Files'.":

  • Settings -> Configuration -> Save Remap Files on Quit: "Save changes to any active input remap file when closing content or quitting RetroArch."

Missing directories in Settings -> Directory

core_options_path

core_options_path = ""

  • Recommended sub-label: "A default path will be assigned if not set. If set, the directory for retroarch-core-options.cfg (requires "Settings -> Configuration -> Use Global Core Options File" enabled [if global_core_options = "true" add: " (done)"]"

Replace in https://github.com/libretro/RetroArch/blob/master/retroarch.cfg

# Path to core options config file.
# This config file is used to expose core-specific options.
# It will be written to by RetroArch.
# A default path will be assigned if not set.
# core_options_path =

with

# Path to core options config file.
# This config file is used to expose core-specific options.
# It will be written to by RetroArch.
# A default path will be assigned if not set.
# This setting depends on
#   global_core_options = "true"
# core_options_path =

video_font_path

video_font_path = ""

# Path to a font used for rendering messages. This path must be defined to enable fonts.
# Do note that the _full_ path of the font is necessary!
# video_font_path =

video_font_path is set to video_font_path = "" for the Flatpak package, and the Appimage.

Document

Comment these variables in https://github.com/libretro/RetroArch/blob/master/retroarch.cfg

bundle_assets_dst_path = ""

  • "Bundle_assets dirs are used, but that should probably not be user configurable."

bundle_assets_src_path = ""`

  • "Bundle_assets dirs are used, but that should probably not be user configurable."

See also

  • https://github.com/libretro/RetroArch/issues/16222

Done

cursor_directory = "/app/share/libretro/database/cursors"

  • https://github.com/libretro/RetroArch/issues/16314

resampler_directory = ""

  • https://github.com/libretro/RetroArch/issues/16314

Replace "DualAnalog" with "Dual Analog":

  • Controls -> Port * Controls -> Device Type: "DualAnalog"
  • Note: To make the "DualAnalog" option visible, you have to load a PlayStation core with a game
  • "DualAnalog is coming from the core(s) directly, so that does not depend on RA". Submitted to https://github.com/OpenEmu/Mednafen-Core/issues/36

Shaders -> Prepend Preset / Append Preset (solved via https://github.com/libretro/RetroArch/pull/16095/files)

  • Both are using identical sub-label "Append preset to the currently loaded preset." which makes it difficult to distinguish them.

Per https://github.com/zoltanvb/RetroArch/commit/ee9e68cb372d2abc07a6facb4a6ee8a3f9b9af3c and https://github.com/libretro/RetroArch/commit/14ce660a38f99d448b32ed752ddaf1f250dcf669 :

  • Settings -> User Interface -> File Browser: "Change file browser settings." <-- s/file browser/File Browser/
  • Settings -> User Interface -> File Browser-> Show Hidden Files and Directories: "Show hidden files and directories in the file browser." <-- s/file browser/File Browser/
  • Settings -> User Interface -> Menu Item Visibility -> [9 entries] : "* (Restart Required on Ozone/XMB)" <-- s/(Restart Required on Ozone/XMB)/ (Restart required on Ozone/XMB)./
  • Settings -> User Interface -> Menu: "Menu driver to use. Requires restart." <-- s/Requires restart/ (Restart required)// -- to indicate Main Menu -> Restart. See more info here.

Per https://github.com/libretro/RetroArch/pull/16144/files

  • Settings -> Directory -> File Browser: "Set Start Directory for the File Browser"

Per https://github.com/libretro/RetroArch/issues/16121

  • Settings -> Input -> RetroPad Binds -> Port 1 Controls -> Save Controller Profile

Bisect Results

[Try to bisect and tell us when this started happening]

Version/Commit

You can find this information under Information/System Information

  • RetroArch: 1.16.0

Environment information

  • OS: Any
  • Compiler: [In case you are running local builds]

davidhedlund avatar Jan 20 '24 19:01 davidhedlund

Main Menu -> Online Updater -> Updater Settings -> On-Demand Thumbnail Downloads: "Automatically download missing thumbnails while browsing playlists" <-- s/playlists/the Playlists directory/
Main Menu -> Online Updater -> Information -> Database Manager: "View databases." <-- s/View databases/View files in the Database directory/

I disagree with these. In neither case are you browsing the directory. You're browsing the contents of the files.

While I like the idea of including the full paths, there's no way most full paths are going to fit in the notification widgets. I suggest including the full path in the logs if they're not already, though.

hizzlekizzle avatar Jan 20 '24 22:01 hizzlekizzle

Main Menu -> Online Updater -> Updater Settings -> On-Demand Thumbnail Downloads: "Automatically download missing thumbnails while browsing playlists" <-- s/playlists/the Playlists directory/
Main Menu -> Online Updater -> Information -> Database Manager: "View databases." <-- s/View databases/View files in the Database directory/

I disagree with these. In neither case are you browsing the directory. You're browsing the contents of the files.

I removed them, thank you for reading through the whole list!

Do you prefer single quotes or not?:

  • Change 'File Browser' settings. or
  • Change File Browser settings.

While I like the idea of including the full paths, there's no way most full paths are going to fit in the notification widgets.

We could add just file names (like for the Online Updater -> Update Assets -> Update * entries) to begin with. Also, the file names are not fully included currently. I just submitted:

  • https://github.com/libretro/RetroArch/issues/16147

davidhedlund avatar Jan 20 '24 23:01 davidhedlund

@zoltanvb Please participate in this issue if you want, since you commited:

  • https://github.com/libretro/RetroArch/pull/16144/files

davidhedlund avatar Jan 20 '24 23:01 davidhedlund

I'd probably go with no quotes, since that sort of thing can be difficult to convey in translations.

hizzlekizzle avatar Jan 21 '24 00:01 hizzlekizzle

While I like the idea of including the full paths, there's no way most full paths are going to fit in the notification widgets. I suggest including the full path in the logs if they're not already, though.

I just submitted this, if you are interested:

  • https://github.com/libretro/RetroArch/issues/16222

davidhedlund avatar Feb 10 '24 22:02 davidhedlund

@zoltanvb I updated all entries in the top post:

  • Made all text significantly more readable.
  • Merged fixed entries under the "Done" section

davidhedlund avatar Feb 28 '24 13:02 davidhedlund

Prepend sublabel was fixed by #16095. Resampler directory seems unused nowadays, can probably be removed from the code. Video font directory gets initialized when browsing for a new font so probably it is not needed to have it configurable separately. Core options path is only used when global core options are enabled (and in that case it points to one file called retroarch-core-options.cfg). Cursor directory is not used any more (views are saved next to playlists), few traces are still in the code that could be removed. Bundle_assets dirs are used, but that should probably not be user configurable. DualAnalog is coming from the core(s) directly, so that does not depend on RA,

zoltanvb avatar Mar 01 '24 15:03 zoltanvb

@zoltanvb I updated all entries in the top post:

* Made all text significantly more readable.

* Merged fixed entries under the "Done" section

I repeated this with your new information. Thank you once again.

davidhedlund avatar Mar 01 '24 20:03 davidhedlund

Merged to the Done section in the top post: Replace "system directory" with "System/BIOS directory"

Thanks!

davidhedlund avatar Mar 12 '24 08:03 davidhedlund

Found that "Maximum Users" requires restart to functioning, which should be added to its sub-label:

Settings -> Input -> Maximum Users Sub-label: "[Current text ] (Restart required)"

davidhedlund avatar Aug 11 '24 21:08 davidhedlund