Brad -

Stripping trailing space is the approach I took in our code to fix the issue - so it definitely works (and is probably the safest way to fix the issue). The reason I took the approach of using getFragment is it seemed that when you do the comparison with checkUrl, it compares one string pulled from window.location with the fragment which has never been added to window.location. So if there were other weird things going on from the DOM side, you might get this issue again.

That being said, this looks like a proper way to fix the issue (and avoid any potential side effects of changing Backbone.history.navigate).