I am not interested in getting rid of ```HeaderRegion``` and neither moving ```sign_in``` method into ```HeaderRegion```  :P 
You are absolutely correct that it doesn't makes sense to define methods that are only ever called once.
I took a different approach to address this issue in next commit, hope you will like it :)