FBX IO: Add file handler #105212

Guillermo Venegas · 2024-03-03T21:43:24+01:00

Guillermo Venegas commented

2024-03-03 21:43:24 +01:00

First-time contributor

Adds file handler to FBX in order to be able to handle drag and drop.

Since panels are unsupported in popups, when the operator is not running
in the space file the import settings are drawing in boxes instead

Adds file handler to FBX in order to be able to handle drag and drop. Since panels are unsupported in popups, when the operator is not running in the space file the import settings are drawing in boxes instead ![image](/attachments/69fa053a-3e1b-45ae-9736-ee236a5eeee4)

image.png

244 KiB

❤️ 1

Guillermo Venegas added 1 commit 2024-03-03 21:43:25 +01:00

add fbx fh c3a1d021f4

Guillermo Venegas requested review from Campbell Barton 2024-03-03 21:45:23 +01:00

Guillermo Venegas requested review from Bastien Montagne 2024-03-03 21:45:24 +01:00

Guillermo Venegas commented

2024-03-03 21:47:40 +01:00

First-time contributor

I know it's maybe a little too late for 4.1, should I change it for main instead?

Jesse Yurkovich commented

2024-03-04 10:52:16 +01:00

This will conflict a bit with my plans to expose FBX as a Collection Exporter - I plan on using layout panels there for instance.

I think we need to fix the popup to support layout panels instead.

This will conflict a bit with my plans to expose FBX as a Collection Exporter - I plan on using layout panels there for instance. I think we need to fix the popup to support layout panels instead.

Bastien Montagne commented

2024-03-04 10:55:55 +01:00

Indeed, would rather make support for panels in popups, than move back to using blocks in UI instead of panels (or even worse - using two different systems depending on the current 'drawing context').

Guillermo Venegas commented

2024-03-04 15:02:09 +01:00

First-time contributor

It's trade-off until support for panels in popups is added.
Maybe just for 4.1 or just as reference if someone wants to
manually apply the patch

It's trade-off until support for panels in popups is added. Maybe just for 4.1 or just as reference if someone wants to manually apply the patch

Daniel Gryningstjerna commented

2024-03-07 13:42:30 +01:00

First-time contributor

Would be great if this landed in 4.1 for now, and then later in 4.2, fix it for real. It would be a bit weird if all other formats supports drag and drop, but not fbx, especially since fbx is the most popular and common format.

Many people would not even consider that drag and drop has been added to Blender if not even the most popular and widely used format is not working. Feels like it would be good to idea to add it in 4.1 for now, to avoid some unnecessary drama, I can already see insane Twitter dram that is gonna unfold from this haha😅

Would be great if this landed in 4.1 for now, and then later in 4.2, fix it for real. It would be a bit weird if all other formats supports drag and drop, but not fbx, especially since fbx is the most popular and common format. Many people would not even consider that drag and drop has been added to Blender if not even the most popular and widely used format is not working. Feels like it would be good to idea to add it in 4.1 for now, to avoid some unnecessary drama, I can already see insane Twitter dram that is gonna unfold from this haha😅

Guillermo Venegas closed this pull request

2024-08-27 22:12:33 +02:00

Guillermo Venegas deleted branch fbx-dnd

2024-08-27 22:12:42 +02:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

No reviewers

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

FBX IO: Add file handler #105212

Pull request closed