Skip to content

Per-widget style#7667

Merged
emilk merged 43 commits intoemilk:mainfrom
AdrienZianne:widget_style
Dec 16, 2025
Merged

Per-widget style#7667
emilk merged 43 commits intoemilk:mainfrom
AdrienZianne:widget_style

Conversation

@AdrienZianne
Copy link
Copy Markdown
Contributor

@AdrienZianne AdrienZianne commented Oct 26, 2025 •
edited
Loading

Implementation of the per-widget style of the action plan to add better styling option to egui

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 26, 2025 •
edited
Loading

Preview available at https://egui-pr-preview.github.io/pr/7667-widgetstyle
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

View snapshot changes at kitdiff

@AdrienZianne AdrienZianne marked this pull request as ready for review November 10, 2025 19:05
lucasmerlin
lucasmerlin requested changes Nov 11, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@lucasmerlin lucasmerlin left a comment

Choose a reason for hiding this comment

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

Nice! Seems like the checkbox is missing in menus for some reason (see the failed snapshot test: https://rerun-io.github.io/kitdiff/?url=https://github.com/emilk/egui/pull/7667)

Comment thread crates/egui/src/widgets/button.rs Outdated
Comment thread crates/egui/src/widgets/button.rs Outdated
Comment thread crates/egui/src/widgets/checkbox.rs Outdated
Comment thread crates/egui/src/widgets/checkbox.rs Outdated
Comment thread crates/egui/src/widgets/checkbox.rs Outdated
Comment thread crates/egui/src/widgets/button.rs Outdated
Comment thread crates/egui/src/widgets/separator.rs Outdated
Comment thread crates/egui/src/widgets/separator.rs Outdated
@AdrienZianne
Copy link
Copy Markdown
Contributor Author

AdrienZianne commented Nov 12, 2025

The check script result in 3 snapshot diff : 2 are the corner of drag value, but Kitdiff show no difference between old/new so I think it's due to linux and x11, and the last diff is in the fractal clock where a single pixel is a bit brighter but again I think it's linux and x11

lucasmerlin
lucasmerlin approved these changes Nov 26, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@lucasmerlin lucasmerlin left a comment

Choose a reason for hiding this comment

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

Awesome, looks good to me now!

@lucasmerlin lucasmerlin added the feature New feature or request label Dec 10, 2025
@emilk emilk added the egui label Dec 16, 2025
@emilk emilk merged commit 9c3a0bb into emilk:main Dec 16, 2025
47 checks passed
This was referenced Jan 31, 2026
Closed
Closed
Masterchef365 pushed a commit to Masterchef365/egui that referenced this pull request Apr 3, 2026
@AdrienZianne @emilk @Masterchef365
Per-widget style (emilk#7667)
c54204e 
* Closes <emilk#7586>

Implementation of the per-widget style of the action plan to add better
styling option to egui

---------

Co-authored-by: adrien <221212@umons.ac.be>
Co-authored-by: Adrien Zianne <adrien@iq002.ipa.iqrypto.com>
Co-authored-by: Emil Ernerfeldt <emil.ernerfeldt@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lucasmerlin lucasmerlin Awaiting requested review from lucasmerlin

Assignees

No one assigned

Labels

egui feature New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Better styling: per-widget styles

3 participants

@AdrienZianne @lucasmerlin @emilk