Conflict with AMP

  • Resolved quattroinviaggio

    (@quattroinviaggio)


    3 years, 11 months ago

    Hi, I’ve tried your plugin and it’s great but I think there’s a conflict with AMP plugin (AMP for WP – Accelerated Mobile Pages for WordPress). Any chance to fix it?

    Thanks
    Francesco

    The page I need help with: [log in to see the link]

Viewing 9 replies - 1 through 9 (of 9 total)
  • Plugin Author CeiKay

    (@ced1870)

    3 years, 11 months ago

    Hi
    Please try to edit the plugin line 107

    function create_tooltip(&$matches) {
$ID = (int) (microtime(true) * 100000); // unique ID

    and modify it like this

    function create_tooltip(&$matches) {
$this->fields = new CKFields($this->options, TOOLTIPCK_SETTINGS_FIELD, $this->default_settings);
$ID = (int) (microtime(true) * 100000); // unique ID

    tell me if it works correctly
    CEd

    Thread Starter quattroinviaggio

    (@quattroinviaggio)

    3 years, 11 months ago

    Hi @ced1870, thanks for answering.
    Unfortunately it didn’t work, as you can see at this page: https://www.quattroinviaggio.it/dubrovnik-arrivo-croazia/amp/

    Thanks
    Francesco

    Plugin Author CeiKay

    (@ced1870)

    3 years, 11 months ago

    please test the 2.2.13 version that shall fix and let me know

    Thread Starter quattroinviaggio

    (@quattroinviaggio)

    3 years, 11 months ago

    Hi, yes the new version works correctly on PC, but on mobile doesn’t work. It shows the {tooltip} marks and the content of the tooltip.
    You can refer to the page https://www.quattroinviaggio.it/dubrovnik-arrivo-croazia/amp/

    Thanks again,
    Francesco

    Plugin Author CeiKay

    (@ced1870)

    3 years, 11 months ago

    yes in a AMP page, it is supposed to have no JS or at least only the minimum
    you can not have the tooltip working in this type of page, the only possibility is to have a minimalist version that shows the tooltip text in a different style …

    I will try that but for sure it will not be the same as on desktop

    Thread Starter quattroinviaggio

    (@quattroinviaggio)

    3 years, 11 months ago

    Having a tooltip instead of the markers would be great!

    Thanks
    Francesco

    Plugin Author CeiKay

    (@ced1870)

    3 years, 11 months ago

    I’m working on a solution to add the text in italic, then you can style it with css
    but the tooltip will not be possible due to the AMP requirement

    Plugin Author CeiKay

    (@ced1870)

    3 years, 11 months ago

    please test the 2.2.14 version which render the tooltip as <i> tag, then you can style it like you want using CSS

    Thread Starter quattroinviaggio

    (@quattroinviaggio)

    3 years, 11 months ago

    Yes! It’s great!
    Thank you very much!

    Francesco

Viewing 9 replies - 1 through 9 (of 9 total)
  • The topic ‘Conflict with AMP’ is closed to new replies.