Change extendable login form directive to instantiated object

I think you are right. This was here so that it could be used by the directive when loading but also to allow inheriting directives access to the settings. I've put the module settings in their own factory now which is a lot cleaner and can be imported by any inheriting directives which need access to the settings.