Skip to content

Fix/155 windows default skin#157

Open
kujirahand wants to merge 9 commits intomainfrom
fix/155-windows-default-skin
Open

Fix/155 windows default skin#157
kujirahand wants to merge 9 commits intomainfrom
fix/155-windows-default-skin

Conversation

@kujirahand
Copy link
Copy Markdown
Owner

@kujirahand kujirahand commented Apr 20, 2026

This pull request improves the theme management for TkEasyGUI, making it easier for users to understand and control widget appearance across platforms. The main changes include setting ttk buttons as the default for a more modern look, providing detailed documentation on theme usage, and adding a test script to compare button styles.

Theme management and defaults:

  • Changed the default for the Button widget to use ttk buttons (use_ttk_buttons=True), resulting in a modern appearance by default. Platform-specific notes and limitations (e.g., background color and height settings) are now documented in the class docstring. [1] [2]
  • Updated theme selection logic and comments to clarify default themes per platform (aqua for macOS, vista for Windows, clam for others) in both set_default_theme and window creation. [1] [2]

Documentation improvements:

  • Added a comprehensive docs/theme.md guide explaining theme usage, available themes, button style options, and customization for TkEasyGUI.
  • Linked the new Theme Guide from the main README for discoverability.
  • Improved the Button widget docstring with usage notes and platform-specific guidance. [1] [2]

Testing:

  • Added a new test script tests/theme/ttk_button.py to demonstrate and compare modern (ttk) and classic (tk) button appearances.

- Change Button use_ttk_buttons default behavior to True on Windows when omitted

- Keep explicit use_ttk_buttons=True/False behavior unchanged

- Keep Windows default theme as vista and document theme usage/customization

- Add docs/theme.md and link it from docs/README.md

- Add tests/theme/ttk_button.py example and lint docstring fix
Copilot AI review requested due to automatic review settings April 20, 2026 07:46
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates TkEasyGUI’s theme/button styling defaults and adds user-facing documentation and a demo script to help compare ttk vs classic tk button rendering across platforms.

Changes:

  • Switches Button to default to ttk.Button rendering (use_ttk_buttons=True) and expands the Button docstring with platform-specific styling limitations.
  • Clarifies default theme selection per platform (aqua on macOS, vista on Windows, clam otherwise) and cleans up related comments.
  • Adds a new Theme Guide (docs/theme.md) and a demo script (tests/theme/ttk_button.py), and links the guide from docs/README.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
tests/theme/ttk_button.py Adds a small demo script to compare ttk vs classic buttons.
docs/theme.md Introduces a Theme Guide covering theme selection and button style options.
docs/README.md Links the new Theme Guide from the documentation index.
TkEasyGUI/widgets_window.py Adds comments clarifying theme defaults and available theme names by OS in root creation.
TkEasyGUI/widgets.py Changes Button(use_ttk_buttons=...) default to True and documents platform limitations.
TkEasyGUI/utils.py Removes stale commented-out theme defaults and keeps Windows default theme as vista.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/theme.md Outdated
Comment thread TkEasyGUI/widgets.py Outdated
Comment thread docs/theme.md Outdated
Comment thread tests/theme/ttk_button.py Outdated
kujirahand and others added 2 commits April 20, 2026 16:50
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
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