Support » Plugin: Responsive Menu » shortcode does not open

Viewing 15 replies - 1 through 15 (of 17 total)
  • Plugin Author Peter Featherstone

    (@peterfeatherstone-1)

    Hi there,

    You have the button twice on your page so it is impossible for it to work.

    This will also invalidate your webpage as there will now be duplicated IDs in your page which could lead to other strange javascript errors.

    Please remove the button you don’t want to use and it will work fine for you.

    You can confirm the HTML errors by checking out your page in the below checker:

    https://validator.w3.org/nu/?doc=http%3A%2F%2Fchikolocksmith.com.s165.upress.link%2Ftest-menu%2F

    Error: Duplicate ID responsive-menu-button.

    Peter

    thank you for the fast reply!!
    ok, how can i remove the top button? via css (display block?) or there is an option for that?

    Plugin Author Peter Featherstone

    (@peterfeatherstone-1)

    Hey Yach,

    What I dont understand is how you have two buttons on the page.

    If you have the Use Shortcode option on then it should turn the normal one off.

    The only way this can happen is if you are calling the shortcode twice in that page?

    Using CSS wont solve your problem as the element will still be in the source code.

    Please advise

    Peter

    ohh found it!! before, i knew i should turn the shortdcode on, i paste the shortcode on the header.php on the child theme, and then another time on a page, so sorry about that. now it works but, i think i have another issue regarding my theme,

    on my header.php if you inspect the element i have 2 headers:
    1. is header with id of #header
    2. is the id #sticky-header

    now i needed to paste the shortcode twice in the header.php in order it to appear on the header and also on the sticky header (that shows after the scrolling)
    and then i run this issue again, because i have 2 shortcode on a page the shortcode on the #sticky-header does not work..

    do you have an idea how to make it work?

    Plugin Author Peter Featherstone

    (@peterfeatherstone-1)

    Hey there,

    I’m afraid that would be a question for your theme developer as I wouldn’t know how the theme works internally etc.

    Can’t you just have the shortcode outside of the 2 headers and control it independently?

    Peter

    i can ask the theme developer what can be done, but as far as i know, there is not way to control them independently,

    why it’s impossible to trigger 2 menus on one page? is it because it’s the free version?
    cause i see now on the pro version here:
    https://responsive.menu/why-go-pro/
    it says: “Responsify two different menus on one page”
    it might help me fix the problem?

    Plugin Author Peter Featherstone

    (@peterfeatherstone-1)

    Hi Yach,

    It’s impossible because then you will have two HTML elements on the page with the same ID which is illegal and will break all sorts of things and make your HTML invalid as per the link I provided above.

    It doesnt make sense to have 2 sets of HTML for one menu on your page either.

    Surely there is a way to put the shortcode outside of the specific header configurations. You could put it in the footer.php file and it should still work.

    Please let me know.

    Peter

    if only the trigger was used by class…
    i can put the shortcode outside the header.php file like the footer, but then how can i call them into the header.php that will appear both on the #header and #sticky-header?
    if you have an idea that would be great

    Here is also the original header.php: https://we.tl/zQzG6hmYb8
    please have a look

    Use a new sticky position CSS option.
    Like this:

    #header {
        position: sticky;
        top: 0;
        z-index: 999999;
    }
    Plugin Author Peter Featherstone

    (@peterfeatherstone-1)

    Hi there,

    Always use IDs where possible, there should only ever be one button so a class doesnt make sense. You also have 2 menus created by having 2 shortcodes so you would be breaking all kinds of duplication rules etc. etc.

    How are you currently using it inside the header? Do you have it set at relative positioning? If so then this will be harder to accomplish. If you are using absolute or fixed then it doesnt matter where it appears in your HTML.

    Please let me know

    Peter

    Dear Moshe my friend!!! thank you so much for the help!! you literally saved me! words can’t describe how thankful i am. also thank Peter for your time and help, Moshe’s trick worked really well

    Plugin Author Peter Featherstone

    (@peterfeatherstone-1)

    Hey Yach,

    Great news and thanks Moshe!

    Have a great day both of you.

    Peter

    Just a note on the browser compatibility for position:sticky
    http://caniuse.com/#feat=css-sticky

    @peterfeatherstone-1 Do you have a git repository of the plugin?
    Maybe I can find some time to help you upgrade the plugin for more fixability 🙂

    Plugin Author Peter Featherstone

    (@peterfeatherstone-1)

    Hey Moshe,

    Sure – please find the repository below:

    https://github.com/peter-featherstone/Responsive-Menu

    Perer

Viewing 15 replies - 1 through 15 (of 17 total)
  • The topic ‘shortcode does not open’ is closed to new replies.