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

Added ability to open explorer at Server World save file location on Launcher. #2033

Conversation

MistaOmega
Copy link
Contributor

@MistaOmega MistaOmega commented May 1, 2023

Addresses #2020 by adding a button to the server worlds menu to open the save file location of that world.
Used existing settings and icons wherever I could to make this addition as seamless as possible.

I also propose replacing the ellipsis icon for the world settings with the cog icon, allowing me to then use the ellipsis icon for opening the save file location which think would make more sense.

Increased the size of each list element to accommodate the new button whilst keeping the old button sizes the same. 83 -> 125 px in height.

@@ -405,7 +427,7 @@ private void TBWorldSeed_Input(object sender, KeyboardFocusChangedEventArgs e)
Config.Seed = TBWorldSeed.Text;
}

private void TBMaxPlayerCap_Input(object sender, System.Windows.Input.KeyboardFocusChangedEventArgs e)
private void TBMaxPlayerCap_Input(object sender, KeyboardFocusChangedEventArgs e)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unsure why, but I think the cleanup profile may have done these changes removing the specific import path. I can add these back if this layout is preferred

Copy link
Member

@dartasen dartasen May 3, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

World manager was developed by one of our support team member (non-developer), so this probably due to adding event with XAML editor

Copy link
Member

@dartasen dartasen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank's for working on this 😀
However we're currently rewriting an MVVM launcher with AvaloniaUI that will allow us to start porting the mod to Linux. So I don't know if we should merge this

Feel freeto join our discord, so we can have a talk about it in dev channels !

@dartasen
Copy link
Member

dartasen commented May 6, 2023

I'm closing this one for now, as discussed on discord

@dartasen dartasen closed this May 6, 2023
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.

2 participants