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

Improve Readability of main + Emphasize E-STOP #12

Merged
merged 4 commits into from
May 8, 2024

Conversation

Astr0clad
Copy link
Contributor

Key Changes:

  • Combines a significant amount of statements in the configurations present in __main__.py for readability
  • Add some whitespace between these statements for readability
  • Make the text of the E-STOP button red and bold to stand out

Additions:

  • Creates the style Quit.TButton in the Application class of __main__.py, used by self.Quit_button

@j5155 j5155 self-requested a review May 8, 2024 07:30
Copy link
Member

@j5155 j5155 left a comment

Choose a reason for hiding this comment

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

Looks great overall and I appreciate how quickly you made a PR!
I just want a few tweaks to the text formatting.

@@ -789,6 +519,7 @@ def __init__(self, root):
style = tkinter.ttk.Style()
style.configure("Red.Label", foreground="red")
style.configure("Green.Label", foreground="green")
style.configure("Quit.TButton", foreground='red', font=('TkDefaultFont', 16, 'bold'))
Copy link
Member

Choose a reason for hiding this comment

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

I think the bold text and font size increase is too much here, just making it red would be fine. I found it a little bit distracting and out of place with the large text and bold.

self.device_info_frame.grid(row=0)
self.device_info_frame.grid(sticky=(N, S, E, W))

self.device_info_frame.config(height=100, width=100)
Copy link
Member

Choose a reason for hiding this comment

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

Code formatting changes look fine to me.

@Iris-TheRainbow
Copy link
Contributor

looks good. merging.

@Iris-TheRainbow Iris-TheRainbow merged commit 3ac028c into unofficial-rev-port:main May 8, 2024
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.

3 participants