-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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.
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
Add some more docs for bevy_text. #9873
Add some more docs for bevy_text. #9873
Conversation
3ad819d
to
cf7630c
Compare
Sorry for the delay in addressing the feedback here. I think I've done it now though. |
cf7630c
to
5ddd715
Compare
By the way, in the future can you make more commits rather than amending and force-pushing when addressing feedback? Not a big deal, but it makes it much easier to check how the feedback was addressed and we squash on merge anyways. |
Just checking in on this to see if there's anything left for me to do. |
Nothing else: let me try and wrangle a second reviewer :) |
# Objective - Have more docs for `bevy_text` to avoid reading the source code for some things. ## Solution - Add some additional docs. ## Changelog - `TextSettings.max_font_atlases` in `bevy_text` has been renamed to ` TextSettings.soft_max_font_atlases`. ## Migration Guide - Usages of `TextSettings.max_font_atlases` from `bevy_text` must be changed to `TextSettings.soft_max_font_atlases`.
# Objective - Have more docs for `bevy_text` to avoid reading the source code for some things. ## Solution - Add some additional docs. ## Changelog - `TextSettings.max_font_atlases` in `bevy_text` has been renamed to ` TextSettings.soft_max_font_atlases`. ## Migration Guide - Usages of `TextSettings.max_font_atlases` from `bevy_text` must be changed to `TextSettings.soft_max_font_atlases`.
Objective
bevy_text
to avoid reading the source code for some things.Solution
Changelog
TextSettings.max_font_atlases
inbevy_text
has been renamed toTextSettings.soft_max_font_atlases
.Migration Guide
TextSettings.max_font_atlases
frombevy_text
must be changed toTextSettings.soft_max_font_atlases
.