Included fxml should can be handled as control.

[bdh92123]

Issue

#319
Included fxml can't be focused, moved, resized.

Fixes

• Included fxml also can be handled in scene builder. So it can also be focused, moved or resized like other node.
• Fixed a bug where the context menu did not appear on included fxml nodes.

[johanvos]

Interesting.
It would be good to have a test that fails before and runs after this patch is applied, can you add some test for this?

[bdh92123]

Hi @johanvos . I uploaded some test in Issue #319 .

However, this fix uses a trick that temporarily treats the FXInstrinsic node as an FXOMInstance node, so it would be correct to fundamentally fix the intrinsic node so that it can also be relocated, resized and focused if it's type is 'fx:include'.

Thanks.

[abhinayagarwal]

Hi @bdh92123,

Thank you for the PR. How big of an effort is it to properly fix this in the Intrinsic node?

[bdh92123]

Hi @abhinayagarwal ,
Scene Builder is designed so that only nodes with bounds (FXOMInstance) can be positioned or resized. Intrinsic node is treated as a completely different class (FXIntrinsic), so if you don't use the trick I did, I think it may require quite a bit of effort.
Thanks.