Skip to content
This repository was archived by the owner on Aug 24, 2024. It is now read-only.

Conversation

@ghanteyyy
Copy link
Contributor

@ghanteyyy ghanteyyy commented Feb 13, 2021

Hello there, I have made the following changes:

[NEW]

  • Merged View UI and Main UI into One UI
  • Added .gitignore file to exclude some files or directories

[MODIFIED]

  • Improved getting full path of images
  • Renamed ui_main.py to main.py
  • Improved adding values to text-widgets
  • Storing values directly in text-widgets
  • Simplified clearing values from text-widgets
  • Improved checking duplicate item before adding it to text-widgets

[FIXED]

  • Fixed automatically adding default values when clicked more than once
    in entry widgets without adding any values
  • Fixed checking if the given value in Value Entry is actually number

[REMOVED]

  • Removed run.py, tips.py, ui_view.py
  • Removed showing quit message before exiting the program
  • Removed variables for storing items, values and explode
  • Removed pop-up messages from each widgets when hovered to them
    because I think the widgets are self explanatory
  • Removed showing message box every time when user inserts a value
    because its annoying to click OK every time.

Important Notes

  • Please delete following files because they are no longer needed:
    • _ _ pycache _ _ directory
    • Instructions.txt, main.png files
    • tips.py, ui_main.py, run.py and ui_view.py
  • And how do you like these changes?

Removed variables for storing items, values and explode
Removed showing message box every time when user inserts a value
Fixed checking if the given value in value_entry is number
Simplified clearing values from text_widgets
Improved adding values to text_widgets
Fixed automatically adding default_values when clicked more than once in entry_widgets when no values are added to them

Deleted unwanted files

Removed unwanted files
Made cursor visible to gif
@nsh07
Copy link
Owner

nsh07 commented Mar 18, 2021

Awesome PR! I'm merging this :)

@nsh07 nsh07 merged commit b789e64 into nsh07:main Mar 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants