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
Editor: import images as planes #28342
Conversation
@mrdoob I'm not sure if this feature is appropriate for a scene editor. What do you think? |
It seems there are merge conflicts now. |
4b8a3ae
to
7d7fca1
Compare
Done. |
It seems the menu point is still missing (see #28342 (comment)). Or maybe it was lost when fixing the merge conflicts? |
Honestly, I am not write this part of the code, because now click the import button directly into the file selector, I do not know how to add this part of the code is more appropriate. |
If we add a submenu directly to the import button, the user experience should be bad, such as adding a submenu, the new user should be unaware that the import button can be clicked. |
Do we make |
I see. Let's just use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you drag a glTF with its bin file and textures into the editor, this happens:
Files for testing: https://github.com/mrdoob/three.js/tree/dev/examples/models/gltf/DamagedHelmet/glTF
Meaning the editor loads all model textures as image planes which is of course not intended.
It seems loading "image as plane" can only work via the menu and not via drag'n'drop.
ae298cb
to
aff2c52
Compare
@Mugen87 I added a new function button entry |
After some more testing I'm not sure its worth adding the feature. I doubt it's super helpful for a scene editor... In any event, I'll defer to @mrdoob . |
Sorry, I don't think we should be doing this. |
Related issue: #XXXX
Description
As the title says.