Making Header Code Does In The Footer Non-AMP Version Link

Resolved birdog123
(@birdog123)

6 years, 11 months ago

Hey Guys,

Likely and very simple adjustment to my code created in the header area or some setting in the plugin to allow this to work properly:

Here is my short explanatory video:
https://www.screencast.com/t/EtMgHUOjy

Here is the code I am using in the header currently:
<p>Click View Non-AMP Version To Load The Full Site Version For Mobile Devices Of This Page.</p>

Thanks guys . . .

🙂

Greg

Viewing 13 replies - 1 through 13 (of 13 total)

Plugin Contributor ampforwp
(@ampforwp)

6 years, 10 months ago

Hey @birdog123,

You have added it incorrectly, it is causing validation errors and Google will not index them. See the REF

If you can tell us where you want to add that code and why, we can help you with that.

Regards
Marqas
Thread Starter birdog123
(@birdog123)

6 years, 10 months ago
Hey there @ampforwp

Right. I noticed that on the javascript version we had before (where it functionally worked but was not amp valid code)…smile.

Here is my very short video explanation along with some further text explanation below too that should now give you a very clear picture:
https://www.screencast.com/t/5ru9yG98Y

So, now we have it this:
<p>Click <a href="?nonamp=1">View Non-AMP Version</a> To Load The Full Site Version For Mobile Devices Of This Page.</p>

**putting that in the amp plugin header section FYI as seen here

So, now it is amp valid code but the “/amp” gets still injected before the “?noamp=1″like this …/amp/?noamp=1 (so it is valid code but does not functionally do the job…hehehe)…

We want all people on notice immediately on the header they can go to our full version to consume content they may not be seeing on the amp version (like tons of external data for data resources, videos, etc which apparently can not be added in amp pages)…we do not want our mobile users confused about where the data they are expecting is and how to go there if they choose.

What is the solution to make it functionally do exactly what the bottom link does and be amp valid code both?

🙂

Greg
- This reply was modified 6 years, 10 months ago by birdog123. Reason: added code piece
Plugin Contributor ampforwp
(@ampforwp)

6 years, 10 months ago

Hey @birdog123,

We already have that option in logo.
Just go to AMP Settings->Homepage-> then Enable “Non-AMP HomePage link in Header and Logo”.

Your logo will take your users from AMP to non-amp site.

Let me know if that helps you or not.

Regards
Marqas

truebird
(@truebird)

6 years, 10 months ago

@ampforwp

I understand the Logo. But we need a more than just that. Actually, we don’t want the logo to do that at all. We need only the clear full sentence I explained in the header that makes it crystal clear to the visitor.

Please let me know . . .
Thread Starter birdog123
(@birdog123)

6 years, 10 months ago
Hey there @ampforwp

Just circling back to be sure you did not miss this:

I understand the Logo. But we do not need or want the logo to go to the non amp version (that would actually be confusing to visitors in our opinion). We don’t want that action in the logo at all.

We need only the clear full sentence I explained in the header that makes it crystal clear to the visitor.

Just need this in our header (we basically have the solution in our code BUT “/amp” injects itself between the current page url and the “/?noamp=1” We just need a solution to where the code or plugin stops doing that and does exactly what the footer link does.

Here is what we need again FYI:
https://wordpress.org/support/topic/making-header-code-does-in-the-footer-non-amp-version-link/#post-9188488

🙂

Greg
- This reply was modified 6 years, 10 months ago by birdog123.
Plugin Contributor ampforwp
(@ampforwp)

6 years, 10 months ago

Hey @birdog123,

I have created a ticket about it in our Github. It will be added in our next updates.

See the ref: https://github.com/ahmedkaludi/accelerated-mobile-pages/issues/892

Thank you very much for your feedback.

Regards
Marqas

Thread Starter birdog123
(@birdog123)

6 years, 10 months ago

@ampforwp

Cool deal. We would still be able to add a bit of our own custom explanation text to it as well right?

But, sounds good…thanks for that…

When do you think the next update will be rolled our roughly?

🙂

Plugin Contributor ampforwp
(@ampforwp)

6 years, 10 months ago

Hey @birdog123,

It will be added in our next update which we are going to release in few days.

Regards
Marqas

Thread Starter birdog123
(@birdog123)

6 years, 10 months ago

@ampforwp I see a update was released. Did it include this? I really hope so . . . smile. If so, how do I implement or adjust to make it work in my code OR how to enable for the header just like the footer etc?

Big smile!!!

🙂

Plugin Contributor ampforwp
(@ampforwp)

6 years, 10 months ago

Hey @birdog123,

Sorry, it is not included in yesterday’s update, but surely it will be added in next updates.

Regards
Marqas

Thread Starter birdog123
(@birdog123)

6 years, 10 months ago

Hey @ampforwp I see another update came through. Is the fix with this one or a earlier one now?

If so, how do I implement or adjust to make it work in my code OR how to enable for the header just like the footer etc?

🙂

Greg

Plugin Contributor ampforwp
(@ampforwp)

6 years, 9 months ago

Hey @birdog123,

I really apologize for all the delays. I was waiting for the update, as we have made a lot of changes in it. So if I would have given you the code, you might have faced some problems with the updation.

Now the update is on and I have created a tutorial as well.
Please check here: Add view non AMP in Header

Do let me know if that helps you or not.

And thanks again for all your patience.

Regards
Marqas

Thread Starter birdog123
(@birdog123)

6 years, 9 months ago

@ampforwp

Ok. I will take a look and report back later.

🙂

Greg

Viewing 13 replies - 1 through 13 (of 13 total)

The topic ‘Making Header Code Does In The Footer Non-AMP Version Link’ is closed to new replies.