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

GUI v9.0 #28

Merged
merged 10 commits into from
Jun 13, 2024
3 changes: 2 additions & 1 deletion launchUI.py
Original file line number Diff line number Diff line change
Expand Up @@ -1566,8 +1566,9 @@ def place_xbar_options(master):
def create_param_pannel(master):
paramPannel = tkinter.LabelFrame(master, text='Configuration', bd=BORDER, relief='groove')
paramPannel.grid(row=0, column=1, rowspan=1, columnspan=1, sticky="nsew")

# Use columnconfigure and rowconfigure to partition the columns, so that each column and row will fill the corresponding space
Copy link
Owner

Choose a reason for hiding this comment

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

nit: period is missing at the end of comment.

Copy link
Author

Choose a reason for hiding this comment

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

Okay. I will add that period.

# The 'weight' represents the weight of the corresponding row/column length.
# The 'weight' represents the weight of the corresponding row/column length.
for i in range(10):
paramPannel.rowconfigure(i, weight=1)
Copy link
Owner

Choose a reason for hiding this comment

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

Please comment on weight.

Copy link
Author

Choose a reason for hiding this comment

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

Okay. Here the entire paramPannel is divided into many rows and columns, and the weights are allocated according to the size of the components to be occupied. The “weight” represents the weight of the corresponding row/column length.

Copy link
Owner

Choose a reason for hiding this comment

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

Ah, i mean add the comment above the corresponding code.

Copy link
Author

Choose a reason for hiding this comment

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

Okay I will do it.

Copy link
Author

Choose a reason for hiding this comment

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

There were some issues with my local environment, and a clash occurred, leading to an inability to sync the changes. I will submit the modified version in the future.

for i in range(3):
Expand Down
Loading