-
Notifications
You must be signed in to change notification settings - Fork 99
Open
Description
really enjoying trawling through your code... i'm giving myself a crash course in backbone/require etc and this is really helping, so thanks heaps for that...
just one small thing.. maybe it's just be not implemented because of time constraints... but when you use backbone's history and press forward and back, shouldn't the menu item 'active' class follow along? that bit just doesn't seem very mvc-ish to me ... i think that small feature would be a really good way to demo how you can decouple that sort of functionality from the ui.. i had a go at fixing it for you, but i'm still haven't got my head completely around the code so i thought best to leave it :)
thanks again! k
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels