Adding amp to links in AMP menu

Your AMP menu is working for me. Most links go to amp pages – a few don’t because I don’t think you have amp versions of those pages or they are being set to hidden. For example the accommodation page.

One thing I do is set up a AMP header and footer menu in the normal wordpress admin panel appearance – menus. I find this is a good way to make sure people stay within the AMP environment by not presenting non-AMP links to them. That may or may not be something you wish to do. It doesn’t look like you have done that.

By using specific menus for AMP it avoids having a mix in the menus of AMP and non-AMP links. A reason for this may be that you have a page which relies on lots of javascript within it – i.e it will not convert to valid AMP so why bother even showing it unless it is a critical type page that must be shown.

• This reply was modified 7 years, 7 months ago by frenchomatic.

Hi
Do not hurry..

It works because i made another menu and added the /amp/ manually, i left some links without because they include some forms and i want visitors to see only the full version.

So, it is better this way because you add /amp/ only to the pages you wish to.

The plugin`s feature dont work(depends from the theme used and many more factors..), but it is not important anymore.

Another point, the plugin created AMP versions for all pages and posts, except home page.
Some pages ofcourse look very funny, they are almost empty since they include forms and lots of javascript…

• This reply was modified 7 years, 7 months ago by satsoklgr.
• This reply was modified 7 years, 7 months ago by satsoklgr.

It isn’t really a case of theme dependency but how you configure your AMP site. That is why it is often best to hide AMP by default on all pages and just activate it on those you want + add specific AMP menus. You get maximum control and flexibility this way. Many people don’t for example use AMP on the home page but some like me do. I prefer to have a complete site in AMP but I don’t have the problems of using javascript on pages – I avoid it to keep things simple, fast and clean.

It is true that AMP kind of forces us to do things in a simple manner and for users in the end it is probably best.

Please logout my adress.

what do you mean? Who is in your address??? I am not part of support and I have no login to your site either.

Hi @satsoklgr,

I just checked and the menu is working fine.

Are you sure that you are still facing some problems with it?
If so, please do let us know.

And also where you want to make CSS changes? Can you share that URL?

Hi Marqas

Yes the menu works fine because i created a new menu and added the /amp/ extention manually, if you read my first answer you will notice this.
And if you saw my first post you was going to know the url…

ok, i just want to change background color in slideshows, make it stand out of the rest content so someone can see that it is a slideshow and not only a picture.

• This reply was modified 7 years, 7 months ago by satsoklgr.

@marqas

There is another prob which i wish you to take a look

Between title and first paragraph there is a huge gap, an empty space.
This is due to the presence of two divs which included an ad in the desktop version.
The ad is not showing , ok, but the space looks strange there and in the AMP source code there are two divs which i like to be removed.

How the div with class “aicp” can be removed? on AMP version only ofcourse

look for this in source code:
<div class=”aicp”><!– Don’t forget to add this div with aicp class –>
<!– auto-article-top –>
</div><!– end of the aicp div –><div class=”kad-spacer clearfix amp-wp-inline-75c36b9967ee40a4b5559e4f7c78ac71″></div></div>

So, in short terms, in a future update give as the possibility to remove full version`s elements by class or id in AMP pages.

Regards

• This reply was modified 7 years, 7 months ago by satsoklgr.
• This reply was modified 7 years, 7 months ago by satsoklgr.

ok, i Just found the way to elimimate visible space by adding this in style.php

.kad-spacer.clearfix.amp-wp-inline-75c36b9967ee40a4b5559e4f7c78ac71 { height:0; }

and this for slider background:

.i-amphtml-fill-content.i-amphtml-replaced-content{ background-color:#eeeeee; }

• This reply was modified 7 years, 7 months ago by satsoklgr.

argggggggggggggggggggrrrrrrrrrrrrr

validation error: The text (CDATA) inside tag ‘style amp-custom’ contains ‘CSS i-amphtml- name prefix’, which is disallowed

it works but gives me validation error

any other way??

fixed.

instead of: .i-amphtml-fill-content{ background-color:#eeeeee; }

i used: amp-img{ background-color:#eeeeee; }

You don’t need to do that in style.php which will get overwritten on upgrade. Try by putting it in the custom css – under the design option.

Hi @satsoklgr,

Please use Custom CSS option present in AMP options panel for additional CSS.

custom css – under the design option doesn`t work

ITS DEAD, there is nothing i can set there, useless, inactive…

Do you want me to count the bags i find everyday? countless,
for instance, how comes and facebook comments work ONLY on new published posts? on the other hand it works in none of the existing posts…

and many more

I know that after an update i ll lose these changes, i keep backup of those.

(i cannot understand how you get everything for granded….and the worse is that you always consider everybody as a novice… thats too bad!)

• This reply was modified 7 years, 7 months ago by satsoklgr.
• This reply was modified 7 years, 7 months ago by satsoklgr.
• This reply was modified 7 years, 7 months ago by satsoklgr.

Hi @satsoklgr,

I apologize if you feel bad. We are just trying to help our users.

And you are facing very strange issues, almost everyone is using Custom CSS option without any bug. And people who are using FB comments are also happy without any issues.

So may be there are chances of conflicting with another plugin or may be with theme.
If you can keep some patience and provide some cooperation then we can surely sort it out together easily.