There is no need to add a list to the `observableA...

# tornadofx

[edvin]

There is no need to add a list to the observableArrayList function. val shapesProperty = SimpleListProperty<ShapeItem>(FXCollections.observableArrayList(arrayListOf())) is redundant. Just do SimpleListProperty<ShapeItem>(FXCollections.observableArrayList()). End result is the same though. More importantly, you have a bug in ShapeItem. The constructor takes two parameters (name and path), but you just throw them on the floor, you don't assign them to your properties.