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

Study separating the base Loader classes for Assets and Locators #9

Open

autumnyard opened this issue May 22, 2021 · 1 comment

Assignees

Labels

enhancement question

Milestone

futher improvements

Owner

autumnyard commented May 22, 2021 •

edited

Loading

SceneLocators shouldn't be available to set as dependency in AssetManager and/or other MapLocator.

autumnyard added enhancement question labels

autumnyard added this to the futher improvements milestone

autumnyard self-assigned this

Owner Author

autumnyard commented May 22, 2021

SceneLocators shouldn't be available to set as dependency in AssetManager and/or other MapLocator.

Or should they? 🤔 Maybe it makes sense for a Map to be loaded inside another. But that should be done as an extension of this plugin, not here.

Should I allow for this extension to be done, even if it allows for this asset to be there?

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