These variables with modal in the name should probably be changed to say dialog instead.  

It gets confusing if we have a different modal directive but the dialog directive also mentions modal in itself.   It should probably be 'dialogClass' instead... although shouldn't the template enforce this class, not the dialog itself?  `<div class="modal">...</div>` in the template.  Or am I missing something?