Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@emilk
Copy link
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

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
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
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
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

3 participants