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

Overhaul #57

Open
wants to merge 16 commits into
base: Custom
Choose a base branch
from
Open

Overhaul #57

wants to merge 16 commits into from

Conversation

burdulixda
Copy link

@burdulixda burdulixda commented Jan 28, 2025

Love this project so much that I decided to give it some much needed love. Appreciate everything you guys have done and I hope my PR gets merged as well.

Changes over version 0.9.2-pre:

Architectural changes:

  • The project now uses Python 3.12 instead of 3.10;
  • The project now uses PySide6 (Qt6) instead of PySide2 (Qt5);
  • Updated all dependencies and tools, including 7-Zip for better security and stability;
  • The project now uses Qt file pickers instead of system-specific ones to improve cross-platform stability.

Design changes:

  • Qt6 allows for much more modern and cross-compatible UI;
  • Users can now switch between Light and Dark themes.

Bug fixes:

  • Fix DLC size calculation (always reported 0KB)
  • Fix not being able to use Open Folder on DLCs

I tested the program extensively and so far I've not encountered any bugs.

Any feedback is appreciated.

image

image

image

@burdulixda burdulixda marked this pull request as draft January 28, 2025 22:14
@burdulixda burdulixda marked this pull request as ready for review January 28, 2025 22:26
@burdulixda
Copy link
Author

Obviously this PR is not yet ready to be merged straight away, I just published it to gather feedback.

@burdulixda
Copy link
Author

@Systemcluster ready for review and merging.

@Systemcluster
Copy link
Owner

Thank you, this is fantastic! I'll look over it in more detail in the coming days. 👍

@burdulixda
Copy link
Author

That's great! I have a lot more where that came from. Already have a list of "nice to haves" from the community and I'll push a new PR over the weekend.

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