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.

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

Updated to Bevy 0.13.0 #36

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Updated to Bevy 0.13.0 #36

wants to merge 1 commit into from

Conversation

nilsegger
Copy link

Hey, using this guide from bevy I updated this library to use bevy 0.13.0.

I ran the examples and they all seem to work.

Some observations that may hint to my changes being faulty:

When running I get the following message:
Duplicate AssetLoader registered for Asset type bevy_text::font::Fontwith extensions["ttf"]. Loader must be specified in a .meta file in order to load assets of this type with these extensions.

For some reason now, even if mesh is Some, meshes.get_mut may return None, maybe this is connected to the above message.

if let Some(mesh) = mesh {
    if let Some(asset_mesh) = meshes.get_mut(mesh) {
        new_mesh = false;
        let ttf2_mesh =
            generate_text_mesh(&text_mesh, &mut font.ttf_font, Some(&mut cache));
        apply_mesh(ttf2_mesh, asset_mesh);
    }
}

I hope my changes are of some use. As said, the examples work at least, so for developing and testing it should be usable.

@Cyannide Cyannide mentioned this pull request Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant