Skip to content

Conversation

@zhangt58
Copy link
Owner

@zhangt58 zhangt58 commented Oct 6, 2025

This is an ongoing work, more examples to be added...

@zhangt58 zhangt58 requested a review from Copilot October 6, 2025 00:10
Copy link

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 adds example files and makes improvements to widget styling and rule functionality. The changes include adding several new example directories with complete working examples, refactoring method names for consistency, and enhancing widget styling capabilities.

  • Adds comprehensive examples for styling, dynamic logic with scripts/rules, and simple widget usage
  • Refactors add_extra_pv to add_pv and set_basic_style to set_style for better naming consistency
  • Enhances styling support for TextEntry and ActionButton widgets with Classic/Native style options

Reviewed Changes

Copilot reviewed 21 out of 28 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
opigen/widgets.py Refactors styling methods and enhances TextEntry widget with ActionWidget inheritance
opigen/rules.py Renames method and improves documentation with better parameter naming
opigen/renderers/phoebus/rules.py Fixes XML node assignment bug
opigen/renderers/init.py Adds safety check for auto_resize functionality
opigen/config/font.def Updates Title font from Liberation Serif to Liberation Sans
opigen/config/attr.toml Adds line_color attribute support
opigen/init.py Adds exception handling for color imports
examples/* Adds multiple example directories with complete working demonstrations

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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.

2 participants