-
Notifications
You must be signed in to change notification settings - Fork 106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FXML #21
Comments
As far as I am aware, Robert is waiting for some JDK bugs to be fixed before continuing this (excellent) work. See this discussion (about half way down): One fork of this project (Kladimir) has added in FXML support which I've just ported to the ClearControl fork which seems to be the most active: https://github.com/SamCooper/DockFX So my recommendation would be to move to ClearControl fork and take my changes. Hopefully they will be merged soon. |
Thanks very much, are there any instructions/demos of how to use it in FXML ? thanks On 20 July 2016 at 14:46, Sam Cooper [email protected] wrote:
|
@kladimir did all the clever work, and updated Roberts demo to use it: https://github.com/Kladimir/DockFX/blob/master/src/main/java/org/dockfx/demo/DockFX.java#L121 Note though, it is FXML support for dock node contents not FXML support for docked node layout. |
I have just figured that out :-) I am looking for FXML support for using the DockPane & DockNode, not simply I have had a go at trying to provide this myself,
--- example use --- <TextArea></TextArea>--- added to DockNode --- public DockNode() {
} --- added to DockPane ---- public List getElements() { @OverRide @OverRide On 21 July 2016 at 10:44, Sam Cooper [email protected] wrote:
|
I would suggest moving this conversation over to: https://github.com/ClearControl/DockFX And re-create the issue there (linking back to this one) as I think this is a subject that @hkmoon is keen on. He seems to have a good idea how this works as he added the tabbed support so may have a better idea what is broken :) If you do that please ref me (@SamCooper) so that I get a notification as this is also something that I am very keen on. |
Are you going to have FXML support in the near future?
The text was updated successfully, but these errors were encountered: