Skip to content

Major Updated and fixes#4

Open
infm5 wants to merge 2 commits into
Warrioravi:masterfrom
infm5:master
Open

Major Updated and fixes#4
infm5 wants to merge 2 commits into
Warrioravi:masterfrom
infm5:master

Conversation

@infm5
Copy link
Copy Markdown

@infm5 infm5 commented Dec 28, 2020

I completely changed the UI and code base of this Calculator

Some MAJOR fixes are:-
-Updated UI

  • included README
  • Included a Icon
    • fixed and updated the code

infm5 added 2 commits December 5, 2020 16:30
made the GUI better with updated code base and fast execution...
- Updated the code 
- Updated the UI
- Included README
Copy link
Copy Markdown
Author

@infm5 infm5 left a comment

Choose a reason for hiding this comment

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

Reviews Done...

Comment thread calculator.py


if __name__ == "__main__":
calculate()
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Overall approach was Nice but The GUI sucks

Comment thread calculator.py
b.pack(side=BOTTOM,padx=3,pady=4,anchor=W)
f1.pack(anchor=S)

root.mainloop()
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

more accurate and charming GUI with fast execution of the Program

Comment thread main.py


# $ Thanks Note
f1 = Frame(root, bg="#EE5537", height=10, width=15)
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

added Frames to make the GUI more attractive

Comment thread main.py
f1.pack(anchor=NW)

# * Third row Buttons
f1 = Frame(root, bg="#EE5537", padx=0, height=20, width=30)
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

added Nice Colours to the Buttons

Comment thread main.py
@@ -0,0 +1,144 @@
from tkinter import *
import tkinter.messagebox as tmb
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Also added tkinter.messagebox for popup notifications

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