I installed first the 1.14 module and then the latest dev version to see if this problem is already fixed, but unfortunately not.

After a fresh install without modifying the text (because this also breaks the module as I use ckeditor - but this is a different ticket!) I see the popup on Windows 10 with IE11. On the same PC with Edge I don´t see the popup. Firefox 47 and Opera 36 also don´t show the popup.

Please fix this as soon as possible as this - and the ckeditor full html problem - makes the module useless at the moment.

brgds Andreas

Comments

Deas created an issue. See original summary.

svenryen’s picture

svenryen CreditAttribution: svenryen commented

Hello Andreas. Do you see any errors in Javascript Console on the browser where it fails?

naveenvalecha’s picture

naveenvalecha
He/Him
CreditAttribution: naveenvalecha at Acquia commented
Status: Active » Postponed (maintainer needs more info)

Please provide more information/steps to reproduce see for more information https://www.drupal.org/contributor-tasks/add-steps-to-reproduce

Ireto Okorodudu’s picture

Ireto Okorodudu CreditAttribution: Ireto Okorodudu as a volunteer commented

I had the same problem with the D8 version. I was using a Chrome version.
The fix was to set the Popup height in pixels value. If left blank the pop up would not appear. When I set it to 75px for example it appeared.

sgonzalez11’s picture

sgonzalez11 CreditAttribution: sgonzalez11 commented

I think the problem is when you put the trailing slash at the end of url in the domain list. For instance: http://local.test/
The code compare the domain list with $base_url variable and remember the $base_url does not include the trailing slash

I removed the slash with the following code and works like a charm

    $domains_list = explode("\n", $domains_list);
    $domains_list = preg_replace('{/$}', '', $domains_list); // I added this line 
    $domain_match = in_array($base_url, $domains_list);

Regards!

  • svenryen committed c9e0453 on 7.x-1.x authored by sgonzalez11 
    Issue #2765843 by svenryen, sgonzalez11: Popup not shown on many...
svenryen’s picture

svenryen CreditAttribution: svenryen at Ramsalt Lab commented
Status: Postponed (maintainer needs more info) » Fixed

Thank you sgonzalez11, I added your patch to -dev.

svenryen’s picture

svenryen CreditAttribution: svenryen at Ramsalt Lab commented
Status: Fixed » Closed (fixed)