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

Separate mod manager logic from GUI #2

Closed
Yato202010 opened this issue May 22, 2022 · 3 comments · Fixed by #6
Closed

Separate mod manager logic from GUI #2

Yato202010 opened this issue May 22, 2022 · 3 comments · Fixed by #6
Labels
enhancement New feature or request

Comments

@Yato202010
Copy link

Yato202010 commented May 22, 2022

It would greatly benefit from separating the mod manager logic from the GUI itself .

  • easier to read
  • you can more easily rework the UI without having to move a bunch of mod manager logic which can increase the chances of making a mistake
  • you can easily make a cli version ( for launching a game without opening the mod manager GUI for exemple, ... )
@MathiewMay
Copy link
Owner

So you would suggest moving all of the logic that has been worked on to rust and pass it to the gui where needed?

@murlakatamenka
Copy link

Yeah, that'd be good. Rust for heavy lifting, Tauri for view and clicking stuff.

@Yato202010
Copy link
Author

Yato202010 commented May 22, 2022

I didn't mean to move the logic to rust but only to separate it from the GUI ( having a logic module which is completely independent from vue )

If you move to rust completely I have a rough prototype in rust of mod manager core which can use overlayfs but very very work in progress with no released date planned. it was mainly for leaning purpose

It's designed as a core library which provides
Trait definition + some helper + simple vfs manipulation and runner library which implement game specific behaviour and how to start the game / load the modlist / handle dependency / ...

If you are interested please contact me on discord at : yato#7998
Since I'm mostly present on this platform

@MathiewMay MathiewMay linked a pull request May 24, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants