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.

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

4519: Mount wads directly under /textures #4548

Merged
merged 2 commits into from
May 26, 2024
Merged

Conversation

kduske
Copy link
Collaborator

@kduske kduske commented May 26, 2024

Wad files used to be mounted under textures/. This was useful
because it allows to filter the wad files in the texture browser
settings, and it also allowed us to show overridden textures in the
browser. At the same time, it complicates handling of texture names
because the texture names cannot be extracted from the file paths in a
uniform matter across different games.

We change this to simplify texture loading, in preparation for the new
material system.

@kduske kduske added this to the 2024.2 milestone May 26, 2024
@kduske kduske force-pushed the 4519-mount-wads-directly branch from c748319 to fa499d6 Compare May 26, 2024 19:22
kduske added 2 commits May 26, 2024 21:44
Wad files used to be mounted under textures/<wadname>. This was useful
because it allows to filter the wad files in the texture browser
settings, and it also allowed us to show overridden textures in the
browser. At the same time, it complicates handling of texture names
because the texture names cannot be extracted from the file paths in a
uniform matter across different games.

We change this to simplify texture loading, in preparation for the new
material system.
@kduske kduske force-pushed the 4519-mount-wads-directly branch from fa499d6 to 6c858c0 Compare May 26, 2024 19:44
@kduske kduske merged commit c65245f into master May 26, 2024
11 checks passed
@kduske kduske deleted the 4519-mount-wads-directly branch May 26, 2024 19:44
@kduske kduske added the release-notes Include this PR in the release notes label Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-notes Include this PR in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant