Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Unify CursorIcon management. #14433

Closed
mintlu8 opened this issue Jul 22, 2024 · 1 comment
Closed

Unify CursorIcon management. #14433

mintlu8 opened this issue Jul 22, 2024 · 1 comment
Labels
A-Windowing Platform-agnostic interface layer to run your app in C-Usability A targeted quality-of-life change that makes Bevy easier to use

Comments

@mintlu8
Copy link
Contributor

mintlu8 commented Jul 22, 2024 •
edited
Loading

What problem does this solve or what need does it fill?

Having multiple sources of CursorIcon management (like bevy_egui) in a project can lead to confusing bugs.

What solution would you like?

Have CursorIcon be reverted by bevy on First based on a resource like DefaultIcon. "Backends" should only write to it if needed to minimize conflicts.

What alternative(s) have you considered?

Keep things as is?
@mintlu8 mintlu8 added C-Feature A new feature, making something new possible S-Needs-Triage This issue needs to be labelled labels Jul 22, 2024
@mintlu8 mintlu8 changed the title Centralized CursorIcon management. Unify CursorIcon management. Jul 22, 2024
@alice-i-cecile alice-i-cecile added A-ECS Entities, components, systems, and events C-Usability A targeted quality-of-life change that makes Bevy easier to use A-Windowing Platform-agnostic interface layer to run your app in and removed C-Feature A new feature, making something new possible S-Needs-Triage This issue needs to be labelled A-ECS Entities, components, systems, and events labels Jul 22, 2024
@alice-i-cecile
Copy link
Member

I think this will be covered with a proper solution to #14351 / #9557.

@alice-i-cecile alice-i-cecile closed this as not planned Won't fix, can't repro, duplicate, stale Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Windowing Platform-agnostic interface layer to run your app in C-Usability A targeted quality-of-life change that makes Bevy easier to use
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@alice-i-cecile @mintlu8