everchron/sochi — pull request #2 by anon-1234 (2014-02-18 09:25:24+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 modal view and have the athlete_modal view extend it, I'd also have a generic modal template and pass what is currently inside the .athlete_modal in athlete_modal.hbs as a partial to it. This would allow for a application wide modal solution that handled any type of child content, and the ability to use the athlete template in other scenarios.