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

Bug : Tour page not adjusted for mobile #138

Open
jagpreetrahi opened this issue Mar 11, 2025 · 5 comments
Open

Bug : Tour page not adjusted for mobile #138

jagpreetrahi opened this issue Mar 11, 2025 · 5 comments
Assignees
Labels
🐛 Bug Something isn't working Priority: Medium This issue may be useful, and needs some attention. Status: In Progress This issue is being worked on, and has someone assigned.

Comments

@jagpreetrahi
Copy link

jagpreetrahi commented Mar 11, 2025

New Description:

please view the below comment.

@JeelRajodiya
Copy link
Member

@jagpreetrahi We currently do not plan to support mobile design - please view this comment. However, we had a redirect screen for the mobile user which would redirect the users to a /mobile page - commit. but this was causing the Search engines to not detect our pages properly, so that commit was reverted.

We can think of adding a new logic which would redirect the users to a /mobile page (or maybe show a popup when the device is mobile) - please keep in mind to stick with the designs for the mobile pages which we had in the previous commits.

@jagpreetrahi Let me know if you like to work on this modified issue, or anyone else who is interested in working on it. Thanks,

@JeelRajodiya JeelRajodiya added Priority: Medium This issue may be useful, and needs some attention. Status: Accepted It's clear what the subject of the issue is about, and what the resolution should be. 🐛 Bug Something isn't working labels Mar 13, 2025
@jagpreetrahi
Copy link
Author

Hey @JeelRajodiya , what I have understand is if we are trying to adding a new logic for /mobile page, it might again still cause search engine issue.
So for that, a better alternative approach could be to dynamically serve the mobile content on the same page without changing the URL.
Or Instead of redirect, a pop up warning users that the site isn’t optimized for mobile and suggesting desktop usage might be a less disruptive approach.

Several SEO issue :
a. Search engine may ignore the desktop page.
b. Duplicate content issue.
c. Loss of page authority.

I'd like to work on this issue, let me know if it's possible

Thanks

@JeelRajodiya
Copy link
Member

@jagpreetrahi The goal is to display a message that "tour is not available on mobile" without affecting indexed pages on google. You can work on it!

@JeelRajodiya JeelRajodiya added Status: In Progress This issue is being worked on, and has someone assigned. and removed Status: Accepted It's clear what the subject of the issue is about, and what the resolution should be. labels Mar 13, 2025
@jagpreetrahi
Copy link
Author

jagpreetrahi commented Mar 19, 2025

hey @JeelRajodiya , as you mention above without affecting indexed pages on google, I have to display a message. For that,
What I am thinking is used meta tag for tell the google like this page is not mobile-friendly
(meta name="robots" content="noindex")
I read some next.js docs, and go through this Next.js , the below implement could be work by adding robots.txt file.
And add : User-agent: Googlebot
Disallow: /mobile
To prevent goggle from indexing /mobile

Looking forward for your opinion
Thanks

@JeelRajodiya
Copy link
Member

sounds good! We can include this in the PR. However it won't resolve the underlying issue of redirecting to the /mobile page while crawling. I am not sure though. Can you think of other ways to prevent the redirection?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working Priority: Medium This issue may be useful, and needs some attention. Status: In Progress This issue is being worked on, and has someone assigned.
Projects
None yet
Development

No branches or pull requests

2 participants