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

Modlist Improvements #13933

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Jade-Harleyy
Copy link

@Jade-Harleyy Jade-Harleyy commented May 7, 2024

This PR:

  • Adds options for renaming and deleting local mods.
  • Adds an option to copy a workshop mod to a local mod.
  • Fixes the context menu's "unsubscribe" button not working.
  • Cleans up context menu creation code, hopefully making it easier to read and work with.
  • Adds methods to the Barotrauma.IO.Directory class to allow for copying and moving directories.

New Localization Tags:

  • RenamePackage - Rename
  • CopyWorkshopToLocal - Copy to local
  • DeleteMods - Delete Local Mods
  • DeleteModsConfirm - Are you sure you want to delete [amount] mod[plural]?\nThis cannot be undone!

- Add options for renaming and deleting local mods.
- Add option to copy a workshop mod to a local mod.
- Fix context menu 'unsubscribe' button not working.
- Cleaned up context menu creation code, hopefully making it easier to read.
@Jade-Harleyy Jade-Harleyy changed the title Modlist Improvements: Modlist Improvements May 7, 2024
@mygamingaccount
Copy link

mygamingaccount commented May 8, 2024

Reviewing:

  • when I try to rename a mod to the same name, it gives me an error (A local package with the name or directory "Modname" already exists) instead of ignoring the change
  • rename text box overlaps with the "Cancel" button

@mygamingaccount
Copy link

I cherry picked this and now it feels awful when Baro updates and this feature is not there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants