?
avatar javimata
javimata
21 Sep 2016

Steps to reproduce the issue

when i open the popup to select some user and i do click in one user the popup close but without select a user, i need open again and select it again and now this work
i use the version 3.6.2

this happend in all sites when need assigned a user like in the author articles, i create a module with this field and have this same bug... i know that this isn't a big problem but i'm work with a site that need use users in many parts, in custom components, modules, etc...

Expected result

i expect work fine 😄

Actual result

System information (as much as possible)

Additional comments

avatar javimata javimata - open - 21 Sep 2016
avatar brianteeman
brianteeman - comment - 21 Sep 2016

I can not confirm this. When I click on the user in the modal popup the
modal is closed and the user is selected

On 21 September 2016 at 18:20, Javi Mata notifications@github.com wrote:

Steps to reproduce the issue

when i open the popup to select some user and i do click in one user the
popup close but without select a user, i need open again and select it
again and now this work
i use the version 3.6.2

this happend in all sites when need assigned a user like in the author
articles, i create a module with this field and have this same bug... i
know that this isn't a big problem but i'm work with a site that need use
users in many parts, in custom components, modules, etc...
Expected result

i expect work fine 😄
Actual result System information (as much as possible) Additional comments

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#12122, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8TIPf33KHP2uQgiz0NvHU5yRK7btks5qsUsggaJpZM4KC72r
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar javimata
javimata - comment - 21 Sep 2016

can do it a search for a user and select?


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/12122.

avatar brianteeman
brianteeman - comment - 21 Sep 2016

yes

On 21 September 2016 at 18:39, Javi Mata notifications@github.com wrote:

can do it a search for a user and select?

This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at issues.joomla.org/joomla-cms/12122
https://issues.joomla.org/tracker/joomla-cms/12122.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#12122 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8S6M2SfsmVLbP1-qiFCVH9TDsk55ks5qsU_NgaJpZM4KC72r
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar dgt41
dgt41 - comment - 21 Sep 2016

Are there any javascript errors logged in the browser's console?

avatar javimata
javimata - comment - 21 Sep 2016

no :-(

sometimes work fine, but many times not

avatar dgt41
dgt41 - comment - 21 Sep 2016

I've just tested it when creating a new article and it works fine.
What is your browser?
Also can you make a screenshot of the scripts loaded in the head? (of course you can comment out the site's url )

avatar javimata
javimata - comment - 21 Sep 2016

i upload the code in the view article

https://gist.github.com/javimata/762104c3e25acc222f64c6aebda0d453

use chrome in mac

avatar alikon
alikon - comment - 21 Sep 2016

please test without any 3dp extensions or template from your gist it seems you are using helix3

avatar dgt41
dgt41 - comment - 21 Sep 2016

I suspect this is the problem

        // Add extra modal close functionality for tinyMCE-based editors
        document.onreadystatechange = function () {
            if (document.readyState == 'interactive' && typeof tinyMCE != 'undefined' && tinyMCE)
            {
                if (typeof window.jModalClose_no_tinyMCE === 'undefined')
                {   
                    window.jModalClose_no_tinyMCE = typeof(jModalClose) == 'function'  ?  jModalClose  :  false;

                    jModalClose = function () {
                        if (window.jModalClose_no_tinyMCE) window.jModalClose_no_tinyMCE.apply(this, arguments);
                        tinyMCE.activeEditor.windowManager.close();
                    };
                }

                if (typeof window.SqueezeBoxClose_no_tinyMCE === 'undefined')
                {
                    if (typeof(SqueezeBox) == 'undefined')  SqueezeBox = {};
                    window.SqueezeBoxClose_no_tinyMCE = typeof(SqueezeBox.close) == 'function'  ?  SqueezeBox.close  :  false;

                    SqueezeBox.close = function () {
                        if (window.SqueezeBoxClose_no_tinyMCE)  window.SqueezeBoxClose_no_tinyMCE.apply(this, arguments);
                        tinyMCE.activeEditor.windowManager.close();
                    };
                }
            }
        };

can you edit /libraries/cms/html/behavior.php and replace that code with

        // Add extra modal close functionality for tinyMCE-based editors
document.addEventListener("DOMContentLoaded", function(event) {
    if (typeof tinyMCE != 'undefined' && tinyMCE)
    {
        if (typeof window.jModalClose_no_tinyMCE === 'undefined')
        {
            window.jModalClose_no_tinyMCE = typeof(jModalClose) == 'function'  ?  jModalClose  :  false;

            jModalClose = function () {
                if (window.jModalClose_no_tinyMCE) window.jModalClose_no_tinyMCE.apply(this, arguments);
                tinyMCE.activeEditor.windowManager.close();
            };
        }

        if (typeof window.SqueezeBoxClose_no_tinyMCE === 'undefined')
        {
            if (typeof(SqueezeBox) == 'undefined')  SqueezeBox = {};
            window.SqueezeBoxClose_no_tinyMCE = typeof(SqueezeBox.close) == 'function'  ?  SqueezeBox.close  :  false;

            SqueezeBox.close = function () {
                if (window.SqueezeBoxClose_no_tinyMCE)  window.SqueezeBoxClose_no_tinyMCE.apply(this, arguments);
                tinyMCE.activeEditor.windowManager.close();
            };
        }
    }
});

Mind that this will not work on IE8, but that could be fixed with joomla 3.7 and the use of polyfills

avatar javimata
javimata - comment - 21 Sep 2016

i do it, obvious, change the " for ' in the second line but not fix it, i have 2 joomla's in the same domain for this you see nuevo in the upload code :-D, in the another dont have helix but in both have the same problem :-S

i upload a video https://youtu.be/2LR0iNu3iH4

avatar andrepereiradasilva
andrepereiradasilva - comment - 21 Sep 2016

confimed in latest staging.
you need to search in the search tools modal to have the issue or just presse "Clear" and them try to select the user

avatar andrepereiradasilva
andrepereiradasilva - comment - 21 Sep 2016

please test #12137

avatar zero-24
zero-24 - comment - 22 Sep 2016

Closing as there is a PR. Thanks!

avatar zero-24 zero-24 - change - 22 Sep 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-09-22 10:30:34
Closed_By zero-24
avatar zero-24 zero-24 - close - 22 Sep 2016

Add a Comment

Login with GitHub to post a comment