Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IE7-9 fails #24

Closed
tom-adsfund opened this issue Feb 21, 2013 · 3 comments
Closed

IE7-9 fails #24

tom-adsfund opened this issue Feb 21, 2013 · 3 comments

Comments

@tom-adsfund
Copy link

Same as tmeasday/meteor-router#33

@alanning
Copy link

@cmather

Now that Tom has an official release of Router supporting IE8+, I'd like to suggest making a similar change to Mini-Pages.

[ Sorry for not submitting a pull request but I'm a bit slammed atm and this was easier for me :-) ]

This would require two changes:

smart.json:

ORIG

  "packages": {
    "page-js": {}
  }

NEW

  "packages": {
    "page-js-ie-support": {}
  }

README.md:

(markup source here: https://raw.github.com/tmeasday/meteor-router/master/README.md)

Internet explorer 8+ support

If you want the router to work in older version of Internet Explorer that don't support pushState, you can use the HTML5-History-API polyfill:

"packages": {
  "HTML5-History-API": {},
  "mini-pages": {}
}

@kenyee
Copy link

kenyee commented Apr 1, 2013

Please add the HTML5-History-API polyfill...I hit this on Debian w/ IceWeasel surprisingly.

Basically, the symptom is that this.redirect doesn't work because it uses Page.js' history.pushState. When you call mini-pages "this.redirect('/blah')" function, it blows up and doesn't seem to do anything.

@cmather
Copy link
Owner

cmather commented Apr 1, 2013

Hey guys, I'm back on the grid now. Sorry for lack of response. I'll push a new version today or tomorrow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants