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

Support custom loading screen music and images #80

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

t0stiman
Copy link
Collaborator

Support custom loading screen music and images and:

  • repository info in MapInfo
  • typos fixed
  • export progress bar improvements
untitled.mp4

- repository info in MapInfo
- typos fixed
- export progress bar improvements
@t0stiman t0stiman marked this pull request as ready for review October 25, 2024 11:29
@t0stiman t0stiman requested a review from Insprill October 25, 2024 11:29
@t0stiman t0stiman mentioned this pull request Oct 25, 2024
@t0stiman
Copy link
Collaborator Author

@Insprill added custom loading screen logo support

image

Validator validator = validators[i];

EditorUtility.DisplayProgressBar("Validating map", validator.GetType().ToString(), i / (float)validators.Length);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's with this change?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I ran into a situation where part of the validation took a long time. This line makes the progress bar show what validator is running so we can see what it's doing.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was already there through, you just moved it below the array access

@@ -28,6 +29,14 @@ protected override IEnumerator<Result> Validate(Scenes scenes)
if (mapInfo.name == Names.DEFAULT_MAP_NAME)
yield return Result.Error($"Your map name cannot be {Names.DEFAULT_MAP_NAME}");

//Loading Screen
//LoadingScreenImages will be null if the map was built with an older version of Mapify
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment doesn't make sense since it's in the exporter. Was it supposed to say something along the lines of LoadingScreenImages will be null if none are configured? I don't recall if Unity would make the array null or empty if it's unset.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment doesn't make sense since it's in the exporter

Not if the map was exported with an older version of Mapify.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm still not following. This is in editor-only code that's only run when you try to export the map. It's never run on an already exported map.

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.

2 participants