[Support request] Generatepress and AMP

Home Forums Support [Support request] Generatepress and AMP

Home Forums Support Generatepress and AMP

Viewing 4 posts - 1 through 4 (of 4 total)
  • Author
    Posts
  • #1620990
    Ivan

    I’m using GeneratePress with wordpress official AMP and a Generatepress AMP solution (beta – I suppose) published at GitHub.

    Generally it works really fine but there is a (minor) problem on smartphones because the sub menu does not show up below the menu as it does on non AMP pages. I have made a work around showing the sub menu in the footer but that is not so elegant.

    I also wonder when the official plugin will be launched at wordpress. I really hope Generatepress (which is the best theme I ever have used) still will support AMP sites.

    Regards
    Ivan
    https://www.jozan.net

    #1621578
    Tom
    Lead Developer
    Lead Developer

    Hi there,

    It should display below the menu. Can you show us a screenshot of the issue as you’re seeing it?

    There are a few more tweaks to be made before uploading it to WP.org.

    #1621741
    Ivan

    Hi Tom – it is a little difiicult to show a screenview from a mobile

    But on AMP pages it looks like this below the header:

    Menu (f.ex https://www.jozan.net)

    and on non AMP pages (f.ex. https://www.jozan.net/newsletter-sign-up/ )

    Menu
    SUBMENU

    Regards
    Ivan

    #1622630
    Tom
    Lead Developer
    Lead Developer

    Ah, are we talking about the secondary navigation?

    As of right now, we rely on some simple javascript to move the secondary navigation after the primary navigation when it exists in the sidebar. It’s not a perfect solution, but it prevents having to render the secondary navigation multiple times on the page and hiding one for the mobile.

    It’s an issue on AMP sites as they don’t allow for javascript. Unfortunately there is no solution as of right now without actually moving your secondary navigation out of the sidebar.

    I’ve made a note of this, as it’s time to move away from that javascript solution.

Viewing 4 posts - 1 through 4 (of 4 total)
  • You must be logged in to reply to this topic.