Skip to content

Conversation

@viridia
Copy link
Contributor

@viridia viridia commented Oct 28, 2025

@ickshonpe This might be useful for managing font atlases (and I have other uses for it as well).
@alice-i-cecile

@github-actions
Copy link
Contributor

Your PR caused a change in the graphical output of an example or rendering test. This might be intentional, but it could also mean that something broke!
You can review it at https://pixel-eagle.com/project/B04F67C0-C054-4A6F-92EC-F599FEC2FD1D?filter=PR-21680

If it's expected, please add the M-Deliberate-Rendering-Change label.

If this change seems unrelated to your PR, you can consider updating your PR to target the latest main branch, either by rebasing or merging main into it.

@ickshonpe
Copy link
Contributor

I'll set up a branch on the font atlases PR and try it out. With font management it's only usually going to ever be a dozen items in the cache max, so performance isn't too important, but it would be better to have a well tested implementation than my adhoc thing.

@ickshonpe ickshonpe self-requested a review October 29, 2025 09:44
@alice-i-cecile alice-i-cecile added A-Utils Utility functions and types S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Oct 29, 2025
@alice-i-cecile
Copy link
Member

I'm not sure that this should go in bevy_platform, since it's not a replacement for an std type. We could just put it in bevy_utils instead 😔 I have no objections to adding a reusable implementation of this though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Utils Utility functions and types S-Needs-Review Needs reviewer attention (from anyone!) to move forward

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants