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

Fix "Full Site" behaviour #10

Open
sminnee opened this issue Mar 31, 2012 · 1 comment
Open

Fix "Full Site" behaviour #10

sminnee opened this issue Mar 31, 2012 · 1 comment

Comments

@sminnee
Copy link
Member

sminnee commented Mar 31, 2012

From http://open.silverstripe.org/ticket/5953

Currently the "Full Site" link doesn't work when redirecting mobile devices to the mobile domain automatically.

A cookie is set, but it doesn't appear to be shared between the full site and the mobile site. Perhaps the domain argument needs to be set for setcookie()?

Would this work if the mobile site was "mysite.mobi" instead of a subdomain "m.mysite.com"?

Automatic redirection behaviour:

When first redirected to mobile site on a mobile device, set a cookie "onMobile" true and another "onFullSite" false to keep track of the state
When full site requested, set another cookie "onFullSite" true and set "onMobile" to false, and check the cookie states to ensure automatic redirects to the mobile site don't occur any longer
When mobile site accessed again after viewing the full site, set "onMobile" true and "onFullSite" false, re-enabling automatic redirects to the mobile site

@mmichelli
Copy link

A suggested change/fix: #31

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

2 participants