Channels
#tornadofx
- p
pim
08/08/2017, 6:09 PM@edvin @jchildress The workaround seems to be working. I've added an explanation comment and will create a PR in a sec👍 1 - e
edvin
08/08/2017, 6:09 PMPerfect, @pim! - e
edvin
08/08/2017, 6:09 PM@pim Add an entry to the changelog if you have time!👍 1 - p
pim
08/08/2017, 6:09 PMStill getting used to contributing 😛👍 1 - e
edvin
08/08/2017, 6:10 PMI'm very happy that you are 🙂 - p
pim
08/08/2017, 6:16 PM@Ruckus
Seems like his JDK is not set up correctly on linuxe: /home/al/project/app/src/pcUi/MyApp.kt: (3, 8): Unresolved reference: javafx - r
Ruckus
08/08/2017, 6:16 PMSo my guess is he's using OpenJDK and didn't install the JavaFX part👍 1 - j
jchildress
08/08/2017, 6:16 PM@pim Yep... I posted that...👍 1 - p
pim
08/08/2017, 6:17 PM@jchildress Did you post an answer? Because it's not visible for me - j
jchildress
08/08/2017, 6:17 PMi did, but i might not have enough credit for it to show? - p
pim
08/08/2017, 6:18 PM@jchildress That's not how SO works, weird - j
jchildress
08/08/2017, 6:18 PMactually mine shows deleted now.. - p
pim
08/08/2017, 6:19 PMAny reason? I can post it again if you want me to - j
jchildress
08/08/2017, 6:19 PMI'm guessing mine was more of a "comment/question", than an "answer"... - j
jchildress
08/08/2017, 6:20 PM@pim go ahead and answer.. thanks.. - p
pim
08/08/2017, 6:20 PMSeems like some moderator went rogue 😐 - r
Ruckus
08/08/2017, 6:21 PMThat's weird - p
pim
08/08/2017, 6:22 PMDoes the openjdk not have jfx? - p
pim
08/08/2017, 6:22 PMNo it does - r
Ruckus
08/08/2017, 6:22 PM@pim It does, but it's not included by default like it is with Oracle. It needs to be added separately. - t
teedee
08/08/2017, 6:23 PMIt doesn't for 8 and lower. I read it was going to be included in JDK 9 but I'm not 100% sure of that - r
Ruckus
08/08/2017, 6:23 PM@teedee That would be nice - p
pim
08/08/2017, 6:25 PMOkay, I posted the answer👍 1 - j
jchildress
08/08/2017, 6:26 PM@pim for the fix do you not want the opposite?
instead...comboBox.widthProperty().onChange { listView.prefWidth = comboBox.width } - p
pim
08/08/2017, 6:26 PM@jchildress Let me check, lol - j
jchildress
08/08/2017, 6:27 PMI actually prefer what you put in, but I'm thinking it's not the same as what @edvin had with 'listView.prefWidthProperty().bind(comboBox.widthProperty())' - p
pim
08/08/2017, 6:27 PMWhat do we want to resize though - j
jchildress
08/08/2017, 6:27 PMSeems like the list view should be the width of the largest item in the combobox choices? at least... - p
pim
08/08/2017, 6:27 PMYou're right, it should be the other way around, woops - p
pim
08/08/2017, 6:28 PMThat's what I was thinking as well, I'm leaving it like this, opinions @edvin ?