Skip to content

Provide ViewModel parameter for ViewLoader #81

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.

Sign up for GitHub

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

Jump to bottom
Closed
sialcasa opened this issue Jul 11, 2014 · 1 comment
Closed

Provide ViewModel parameter for ViewLoader #81

sialcasa opened this issue Jul 11, 2014 · 1 comment
Assignees
@manuel-mauky
Labels
API breaking change Feature
Milestone
0.3.0

Comments

@sialcasa
Copy link
Owner

loadViewTuple in ViewLoader should enable the user to provide an own viewmodel for the creation of the view
@sialcasa sialcasa added this to the 0.3.0 milestone Jul 11, 2014
@sialcasa sialcasa changed the title Provide ViewModel parameter for Provide ViewModel parameter for ViewLoader Jul 11, 2014
@sialcasa
Copy link
Owner Author

Reuse of viewmodels to change the representation

@manuel-mauky manuel-mauky mentioned this issue Aug 6, 2014
Closed
manuel-mauky added a commit that referenced this issue Aug 11, 2014
@manuel-mauky
#81 FluentViewLoader now uses FxmlViewLoader and JavaViewLoader direc…
fc1945b 
…tly instead of using ViewLoader.
manuel-mauky added a commit that referenced this issue Aug 11, 2014
@manuel-mauky
#81 Added a test case for embedded sub views. This test verifies that…
5c74b22 
… sub views are also correctly initialized and the viewModels are injected.
manuel-mauky added a commit that referenced this issue Aug 11, 2014
@manuel-mauky
#81 Refactoring of the DepenencyInjector, implemented the possibility…
47f4f34 
… to use existing ViewModel instances for FxmlViews

- Extracted the functionality for injecting ViewModels from the DependencyInjector class into it's own class: ReflectionUtils.
manuel-mauky added a commit that referenced this issue Aug 11, 2014
@manuel-mauky
#81 JavaViews can now use existing ViewModels too
475bcc7
manuel-mauky added a commit that referenced this issue Aug 11, 2014
@manuel-mauky
Merge pull request #112 from sialcasa/#81_viewmodel_param
85b7785 
#81 viewmodel param
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@manuel-mauky manuel-mauky

Labels
API breaking change Feature
Projects
None yet
Milestone
0.3.0
Development

No branches or pull requests
2 participants
@manuel-mauky @sialcasa