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

The most significant changes involve the modification of the renaming… #2256

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JanPanthera
Copy link

… method in the CTkTabview class and the addition of code to update the _current_name attribute. These changes ensure that the tab renaming process is more efficient and that the connection to the frame is maintained even when a tab is renamed.

Changes:

  1. The renaming method in the CTkTabview class has been altered. Instead of removing the old name from the _name_list and appending the new name at the end, the new name now replaces the old name at the same index in the _name_list. This change makes the renaming process more efficient.
  2. Code has been added to update the _current_name attribute if the old name matches the current name. This is a crucial change as it ensures that the connection to the frame is not lost when a tab is renamed.

… method in the `CTkTabview` class and the addition of code to update the `_current_name` attribute. These changes ensure that the tab renaming process is more efficient and that the connection to the frame is maintained even when a tab is renamed.

Changes:
1. The renaming method in the `CTkTabview` class has been altered. Instead of removing the old name from the `_name_list` and appending the new name at the end, the new name now replaces the old name at the same index in the `_name_list`. This change makes the renaming process more efficient.
2. Code has been added to update the `_current_name` attribute if the old name matches the current name. This is a crucial change as it ensures that the connection to the frame is not lost when a tab is renamed.
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