hi all, good (Australian) evenig.

just starting out with tornadofx, and I'm having a small issue with a nested view, and I was wondering if someone could help me out.

@User uploaded a file: Untitled and commented: The problem is line 31 and 32. If I inline the textarea, it works. If I have the textarea in a separate class, it doesn't, so I'm doing something wrong... but what?

textarea is not a container

wrap it in a vbox or hbox

@edvin if you're looking for suggestions on #196, i work on an app that solved this with a custom Google Guice scope Use an annotation that's neither -at-Singleton nor -at-Prototype, say -at-Project if you were building something like Intellij IDEA or -at-Document for something like the Microsoft Word. Then, any other -at-Project annotated classes will be injected in the proper scope when an object graph is started like from a Stage This lets you use "by inject()" the same way you're using the Controller injection, but all the components in a given scope will have their proper references, not an app-wide Singleton or an unshared Prototype

@ron it's wrapped in a container, right?

it's wrapped in the container in EditorView

I dont think it works that way

well tbh I always solve it by putting it in a vbox

well, my label also is not in a vbox, and that does work

and if I put my textarea in a vbox, it still doesn't display

hmmz then I am afraid you have to wait for edvin

(I think that if I put my textarea in a vbox, which is already in a vbox, that it won't work well)

Ah, I fixed it!

@User uploaded a file: Untitled

Changing EditorView to have a root BorderPane works.

@ron thanks for the help, much appreciated!

Np

@carlw I already implemented and committed a solution to #196 earlier today 🙂

@bodiam The problem you're facing is that you simply mention a variable, you don't assign it to anything 🙂

textarea("this works")

calls the

textarea

builder. It creates a textarea and adds it to the children list of the parent (root in your case) - that's why it works 🙂 Doing

outputView.root

simply says "variable". It does nothing. You need to assign it do the children list. It can be done in many ways, the simplest is just

this += inputView

. If you don't use

inputView

for anything else, you don't even need to inject it, you can just say

this += (InputView::class)

and it will be automatically injected.

@bodiam the

+=

thing works for nodes also, so you could say

this += inputView.root

, but there is an overloaded version that takes a View, that's why

this += inputView

works as well.

the

+= (InputView::class)

only works with the plusasignment?

@ron What do you mean?

as

override val root = borderpane {left = (CategoriesListView::class)

does not work

(I was hoping that would work too ;)

@ron That's right. The issue with

=

is that it requires a certain type, and that type is not KClass 🙂

I know, but one could hope

Hehe 🙂

@edvin Cheers, thanks, that's clear!