Skip to content

Remove accesskit feature and always depend on accesskit#7701

Merged
emilk merged 4 commits intomainfrom
emilk/remove-accesskit-feature
Nov 12, 2025
Merged

Remove accesskit feature and always depend on accesskit#7701
emilk merged 4 commits intomainfrom
emilk/remove-accesskit-feature

Conversation

@emilk
Copy link
Copy Markdown
Owner

@emilk emilk commented Nov 11, 2025

With this, egui will always depend on accesskit, removing a lot of #[cfg(feature = "accesskit")] throughout the code.

@emilk emilk added dependencies Pull requests that update a dependency file egui labels Nov 11, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 11, 2025

Preview available at https://egui-pr-preview.github.io/pr/7701-emilkremove-accesskit-feature
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

@emilk emilk force-pushed the emilk/remove-accesskit-feature branch from 783831a to e17eb4a Compare November 11, 2025 19:44
@emilk emilk force-pushed the emilk/remove-accesskit-feature branch from e17eb4a to f10949f Compare November 11, 2025 19:45
@emilk emilk requested a review from lucasmerlin as a code owner November 11, 2025 20:15
@alltheseas alltheseas mentioned this pull request Nov 11, 2025
1 task
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.

I guess the only bad thing about this is that it will further bloat the wasm bundle sizes (even tough accesskit doesn't even support wasm).

@emilk
Copy link
Copy Markdown
Owner Author

emilk commented Nov 12, 2025

It indeed adds almost 100kB, from 7 367 kB to 7 451 kB, but I think its worth it not to have all those feature flags

@emilk emilk merged commit 1af5d1d into main Nov 12, 2025
46 of 47 checks passed
@emilk emilk deleted the emilk/remove-accesskit-feature branch November 12, 2025 09:51
lucasmerlin added a commit that referenced this pull request Nov 19, 2025
Masterchef365 pushed a commit to Masterchef365/egui that referenced this pull request Apr 3, 2026
* Closes emilk#3137

With this, `egui` will always depend on `accesskit`, removing a lot of
`#[cfg(feature = "accesskit")]` throughout the code.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file egui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove accesskit feature in egui, and always depend on it

2 participants