mbrlabs / Mundus

A 3D world/level editor built with Java, Kotlin & libGDX.
Apache License 2.0
214 stars 36 forks source link

Uber bug with assets. #31

Closed krukhlis closed 8 years ago

krukhlis commented 8 years ago

Ok, so the latest version has complitely redesigned assets manager. Which comes with lots of issues. Please see the image attached. textures_loading_management_issues

In short: 1) The "Assets" menu lets you load only models, not textures. 2) To load texture one can use only submenu of the button before "Gift box button". 3) One can load the same texture multiple times which is waste of the resources. 4) Assets outline shows only models, not textures. :(

mbrlabs commented 8 years ago

1) The Assets menu in the menu bar does nothing, it shows all assets unfiltered for debugging reasons 2) Yes, you can import new textures and models there 3) What do you mean multiple times. Sure, you can import the same asset multiple times if you choose. But the asset manager loads the resource only once if you open the project. 4) Yes, it shows only models. That's intended for now. Later on it will act as asset browser that shows all assets in a directory tree like a file explorer.

A few more notes:

krukhlis commented 8 years ago

Regarding 3) So I can import multiple times the same texture. And in assets browser I will see multiple lines with this texture: mundus-assets1 And I can use any of this as texture for my model. However, if save the project and reload Mundus -- I see the texture was loaded only once and I have only 1 entyry in assets browser: mundus-assets2

Which is inconsistent with the original situation. :)

mbrlabs commented 8 years ago

Oh yes, that's a bug ;) It's overriding the files when importing the asset for the second time. That's why they are gone after the restart. We need to check if there already exists a file with that name and if yes, add an extension to the filename. Or just forbid to import a new asset with the same name. I'd prefer the second option, because it better reflects the organistion in folders.

I'll do that tomorrow.

krukhlis commented 8 years ago

I also agree with the second option, cool!

mbrlabs commented 8 years ago

Alright, i fixed the bug by preventing the user to import multiple assets with the same name.