@carlw It works like a charm

great. @edvin is recommending the action {} style instead of the setOnAction {}.

yeah I know even though I think we should have called it

onAction

😆😜

I think we also concluded that if you only supply an action, and don't configure a button in any other way, it looks clean to do

button("Click me").action { doSomething() }

.

Luckily, if you prefer

onAction

, it's super cheap to add @nimakro 🙂

Just a one liner in your project 🙂

@edvin Hehe I know I was just teasing you 😉

@carlw I think you need curlies in your string template here:

println("prop $toggleTestProp.value")

It must be

println("prop ${toggleTestProp.value}")

, mustn't it?

Yes and since @edvin started already @carlw there is one

>

to much

val toggleTestProp = SimpleObjectProperty <ToggleTestEnum>>()

@nimakro that's an HTML conversion error

@edvin yeah. you'll get the toString() plus the constant ".value"

@User uploaded a file: Untitled and commented: So I got the following code I want that the toggelProperty reflects the current selected button, but I don't want that the toggelbutton calls the

toString

on the enum since the visual representation is done by the graphic. To prevent the

value.toString()

I am doing this `togglebutton("", value = it) is ther a better way

It looks like you're just doing

togglebutton(value = it)

, so you don't assign anything to the text?

@edvin Yeah that's what I want to do because the text is constructed in the

graphic

, but I guess I just have to do `togglebutton("", value = it) then everything works like it should.

Ah, now I understand. If you don't do "", then the value property is automatically assigned. I did that as kind of a fallback, so you don't need to specify the text if the value has a proper tostring.

So if you want to "opt out" of that, you need to assign it some value, like you do.

Let me check the code.

@edvin Yeah sorry for my complicated explanation, yes that exactly what you did..

I think it's fine I will just do

""

because, I think the fallback is wanted in 99% of the time

I guess you can do

togglebutton(value = it) { text = null }

. It's a bit more explicit perhaps, but I understand that it might not feel elegant. I tried to be smart by giving it the default value, but it backfired for you now 🙂

@nimakro That was my hope, that the fallback was a good default 🙂

@edvin Don't worry it's no big deal to do

""

I just wanted to make sure that I am doing things the right way 😉

@nimakro Your code looks good 🙂 What are you creating?

@edvin A client for

Todoist

as an showcase for

TornadoFx

, this project is part of an assignment I have to do for my university I will post some pictures as soon as I have something presentable... and I will open up the repo as well.... later on the client will also integrate with github to track issues ... the main idea is to have a program, to manage all the tasks in one place and with offline capabilities.

Cool! We're here if you get stuck on anything, but I doubt you will 🙂

@edvin Thanks we will see I am just so slow with UI-Stuff 🙈 You would probably be done by now 😉

@edvin how was the frog?

@kastork It was OK, but not mind blowing. The duck was much better 😁

That's something I guess