Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Split app key from viewport title to stop spraying app.ron files everywhere #59

Merged
merged 1 commit into from
Aug 6, 2024
Merged

Split app key from viewport title to stop spraying app.ron files everywhere #59

merged 1 commit into from
Aug 6, 2024

Commits on Aug 6, 2024

  1. Split app key from viewport title to stop spraying app.ron files ever… 

    …ywhere.

Egui uses the app key to determine the storage location for persisted
configuration files. This does not work well when the app key is
something like "~/Downloads/prof_0.gz - Legion Prof" (the case when
profiling files you've just downloaded), which results in the app.ron
files being persisted to some random subdirectory of ~/Downloads
instead of "~/Library/Application Support/Legion Prof" or something
similar (depending on the OS).

So split the app key off from the viewport title so that the former
can be short and predictable and not mess with the persistent config
location so badly.
    @elliottslaughter
    elliottslaughter committed Aug 6, 2024
    Configuration menu
    Copy the full SHA
    ac1e5e1 View commit details
    Browse the repository at this point in the history