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

Improvements to the main project and the GUI #163

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

RatserX
Copy link

@RatserX RatserX commented Feb 26, 2024

Here's a couple of improvements I did, mostly to the GUI:

  • Project:
    • Add a .gitignore file to avoid pushing certain Python related files and folders, this includes the config.json file too.
    • Add a requirements.txt file to easily install all related dependencies (updated the README file to mention this).
  • GUI:
    • Avoid O(n*2) loop when processing player info.
    • Sort player names in alphabetical order.
    • Set fixed window size to 350x250.
    • Disable option to resize window.
    • Increase width of comboboxes so that all names fit correctly.
    • Fix external save files showing as blank in the comboboxes, showing the filename GUID instead.
    • Add spacing between group of elements.

Here's how the GUI window currently looks like with these changes:
image

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.

1 participant