Well, one way to do it was using the angular-ui bootstrap which was pretty nice for a first shot, but I didn't like having so much in the controller.
Another problem that I faced was when modifying something in the Modal window, it was immediately reflected back on my original object. This was because a two-way binding was set between the two. This is easily fixed by using angular copy.
Then I started looking into how I could move all this code inside a directive so that everything can be re-used. I managed that, but the angular copy didn't go away.
Having to angular copy everything wasn't really something I did like so much.
I've started looking into the angular code and found that with some little changes I could add a whole new operator besides @, =, &; one that could do exactly what I need it to do: one-way-binding to an object.
I've submitted the code on GitHub and the guys were very helpful in sharing some of their ideas on the matter.(if you wonder why the CI build failed, well, I implemented and tested everything on v1.2.16 and ported the code to the current version and it seemed like I miss typed something in the regex expression. I wasn't about to commit that right away as much as I was looking for some input on the idea)
Based on that input I've come up with a whole new approach to the problem. You can check it up on plnkr.co where a functional example is provided.
As you can see in the example, in the index.html file, I'm using two directives, one named modal that will handle the Modal window and it can be used globally so we don't need to write the same code all over again, and the second directive that handles the content itself, which can be quite anything.
This has been taken and modified from a stack overflow answer in which S Hasan was so helpful to also provide a working example.
If you are wondering why the on-hide function looks like that
it is because I didn't want to pollute the controller with unnecessary code. This is the same as well for all other places I've used this approach.
Now if we were to refer to:
The one-way-bind-person is a naming convention based on one-way-bind- and entity name which we want to one-way bind into the directive.
The on-save is going to be the function that gets triggered when the OK button is clicked and mainly the save is happening and we can use as below:
The onSave function is called in order to return the function vm.savePerson that was passed in so that it can be called with the person parameter.
The person parameter is an object on the directive's scope which is added in the directive's link function:
This is also the place where the one-way binding happens. The parentValueWatch function returns the reference to the parent value, vm.selectedPerson, and when that changes, the onParentValueChange function is going to copy the contents of the vm.selectedPerson over to the directive's scope object person .
I am watching for a reference change only(if you need to watch for the properties of an object or if you pass in an array, then use $watchCollection)
- My working example
- Special thanks to S Hasan for providing a great stack overflow answer with a working example
- Special thanks to the guys from GitHub for providing an awesome feedback: gkalpak and lgalfaso
- Would be nice to have an extension point in AngularJS in which we could define our own operators so that less and more organized code could be written.