everchron/sochi — pull request #3 by anon-1234 (2014-02-18 10:05:41+00:00)
Title: Athletes modal with full country display and formatted sport text

Some notes on the approach:

1 - Built for Chrome, tested in latest FF and Safari - should work everywhere but will lose Modal transitions and have some styling issues on old browsers
2 - Opted to introduce use of Thorax.LayoutView for the modal in order to leverage the built in view management aspects
3 - Made a simple custom modal interface rather than bringing in a 3rd party system
4 - Thorax auto template loading did not appear to work as per the docs, but I found a reasonable way of making it work

Further thoughts:

Taking this further, I'd probably build a base SochiModal view and have AthleteModal view extend it, I'd also have a generic modal template and pass what is currently inside the div.athlete_modal in athlete_modal.hbs as a partial. This would allow for a application wide modal solution that handled any type of child content, and also give the ability to use the athlete template outside of a modal.