-
Notifications
You must be signed in to change notification settings - Fork 602
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
4.0.0 router can't use /
inside of an iframe
#344
Comments
Oh that's interesting - not entirely sure what's causing that - I think it might because of how we initialize our base route in https://github.com/yoshuawuyts/sheet-router/blob/master/create-location.js. The 4.x routing patch was intended to make apps / mobile / electron easier to use, I hadn't really considered What might work (untested) is setting a base tag so it resolves correctly - https://developer.mozilla.org/en/docs/Web/HTML/Element/base - could you let me know if that perhaps works? |
No luck using the
loads the script, but gives me the same error as before. |
Some tools moarn about base-tag not being a full-qualified URL (with protocol, subdomain and domain). Although absolute or even relative paths are valid according to the spec. IIRC there were some differences between |
Released https://github.com/yoshuawuyts/microcomponent recently; think we're close to getting a good set of choo modules wired up in a standalone package aimed to be integrated in other libs. Most things don't need a router, so might as well make it standalone. Sooon ✨ edit: linked to the wrong package |
So |
should this be closed now that choo@6 is out? |
@goto-bus-stop yep I think so. Closing! |
4.0:
http://plnkr.co/edit/HsO6Dwtv1L4CDYi10Gv9?p=preview
3.0:
http://plnkr.co/edit/KSLgxEQveieLKWo5okxt?p=preview
In 3.0,
choo
properly detected the "base" route inside an iframe as/
. Now in 4.0, it's forcing me to use thedocument.location.pathname
of the iframe.I would prefer to use
/
inside of iframes, but I don't know enough about routing to know if this is a bug/limitation/whatever.The text was updated successfully, but these errors were encountered: