Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Remove set_theme function hack #107

Closed
rdbende opened this issue Jul 24, 2023 · 0 comments · Fixed by #113
Closed

Remove set_theme function hack #107

rdbende opened this issue Jul 24, 2023 · 0 comments · Fixed by #113
Labels
high priority

Comments

@rdbende
Copy link
Owner

rdbende commented Jul 24, 2023

The script must configure some colors after the theme is set, but can't do a bind . <<ThemeChanged>> because it is invoked for each widget. I wonder though if bind [winfo class .] <<ThemeChanged>> would work.
@rdbende rdbende added enhancement New feature or request high priority labels Jul 24, 2023
@rdbende rdbende pinned this issue Jul 24, 2023
@rdbende rdbende removed the enhancement New feature or request label Jul 24, 2023
@rdbende rdbende mentioned this issue Jul 26, 2023
Merged
@rdbende rdbende linked a pull request Jul 26, 2023 that will close this issue
Get rid of hacky set_theme proc #113
Merged
@rdbende rdbende unpinned this issue Oct 13, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
high priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Get rid of hacky set_theme proc rdbende/Sun-Valley-ttk-theme
1 participant
@rdbende