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

Use HTML5 pushstate #30

Closed
pstoica opened this issue Jun 6, 2013 · 6 comments
Closed

Use HTML5 pushstate #30

pstoica opened this issue Jun 6, 2013 · 6 comments

Comments

@pstoica
Copy link
Contributor

pstoica commented Jun 6, 2013

Would everyone be alright with putting the JSON controllers in /api/ so we can use HTML5 pushstate instead of hashbang URLs?

@jakswa
Copy link
Contributor

jakswa commented Jun 6, 2013

Do we know what browsers our audience will be using? Hopefully they, too,
know that IE is to be avoided (IE8 and below, for sure)...

On Thu, Jun 6, 2013 at 1:29 PM, Patrick Stoica notifications@github.comwrote:

Would everyone be alright with putting the JSON controllers in /api/ so we
can use HTML5 pushstate instead of hashbang URLs?


Reply to this email directly or view it on GitHubhttps://github.com//issues/30
.

@pstoica
Copy link
Contributor Author

pstoica commented Jun 6, 2013

Angular provides a fallback for older browsers if you use html5mode. Only problem right now is that routes are kind of clashing.

@jakswa
Copy link
Contributor

jakswa commented Jun 6, 2013

Oh, nice, I'm all for that, then. We'll just need to make sure that any
server-side URL rewriting by rails happens if needed. "Using HTML5 mode
requires URL rewriting on server side, basically you have to rewrite all
your links to entry point of your application (e.g. index.html)."
-linkhttp://docs.angularjs.org/guide/dev_guide.services.$location#htmllinkrewriting

On Thu, Jun 6, 2013 at 3:11 PM, Patrick Stoica notifications@github.comwrote:

Angular provides a fallback for older browsers if you use html5mode. Only
problem right now is that routes are kind of clashing.


Reply to this email directly or view it on GitHubhttps://github.com//issues/30#issuecomment-19067435
.

@pstoica
Copy link
Contributor Author

pstoica commented Jun 6, 2013

In that case we can just set up routes in the /api/ namespace and redirect anything else to root (Devise will also be moved over to Angular, so that could also go in /api/). Seem good?

@v3n
Copy link
Contributor

v3n commented Jun 6, 2013

+1 from me.

On Jun 6, 2013, at 4:45 PM, Patrick Stoica notifications@github.com wrote:

In that case we can just set up routes in the /api/ namespace and redirect anything else to root (Devise will also be moved over to Angular, so that could also go in /api/). Seem good?


Reply to this email directly or view it on GitHub.

@jamesdabbs
Copy link
Member

No longer relevant. See #84.

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