That does look better.  I updated the pull request to match.  Thanks.  

Have a great day!

Marshall Thompson
Creative Ideal


On Wednesday, January 23, 2013 at 1:38 PM, David Luecke wrote:

> In route/pushstate/pushstate.js:
> > @@ -34,14 +34,17 @@ steal('can/util', 'can/route', function(can) { > _setup: function() { > // intercept routable links > can.$('body').on('click', 'a', function(e) { > - if(can.route.updateWith(this.pathname+this.search)) { > + // Fix for ie showing blank host, but blank host means current host. > + this.host == '' ? this.host = window.location.host : null;  
> This looks a little weird. Isn't this just doing
> if(!this.host) { this.host = window.location.host; }  
>  
> Besides that I'd merge this in because it makes sense. I was thinking about a good way for testing this but I don't think there is (even checking the window.location in an iframe wouldn't really tell us anything).
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/bitovi/canjs/pull/249/files#r2748417).  
>  
>  
>