Right now the dropdown does not work

Front end? Tag field?

Tags component in the backend. Edit a tag and try to select a parent tag.Am 31.12.2018 01:19 schrieb dGrammatiko notifications@github.com:
Right now the dropdown does not work

Front end? Tag field?

—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.

Looks good for me. But do not see the error in the tag field either.

Ok, explaining the issue with the tags field again in detail:

• Current head of 4.0-dev with blog sample data and debugging enabled
• Creating one tag to have a parent.
• Creating a second tag to select as a child of the previous parent.
• When creating that second tag, I can't select an existing tag as parent. The tags are retrieved and displayed, but I can't select them. I also get JS errors in the console. (FF64)
• On a quick check, it seems as if the same issue is in the article edit view.

but I can't select them

I am guessing it's from: #22263
That PR has numerous issues:

• Custom elements cannot rely on other scripts being already loaded (this works right now because I'm lazy loading the custom elements, but you could load them asynchronously and then they'll fail)
• It doesn't use mutation observers to mutate any added/removed options from the select (or the select itself)
• It doesn't refresh correctly: in the article edit (admin) try to resize to mobile and observe the tags field, values disappear
• Numerous accessibility issues mainly because choices itself is not accessible (and probably not fixable in Joomla's repo).

@Fedik can you please check those issues?

I don't know what the issue is, but I'm unable to select a parent for a new tag.

I just checked and it works fine for me in Chrome and Firefox, without errors in console.

@Hackwar which exactly error did you got?

@dgrammatiko

Custom elements cannot rely on other scripts being already loaded (this works right now because ...

If it works then all fine ?

It doesn't use mutation observers...

because it does not need

It doesn't refresh correctly

The mobile styling issue I better leave to someone who better know the template styles ?

choices itself is not accessible

hm? I thought it accessible enough, that why we pick that script

@Fedik TypeError: e.replace is not a function in choices.min.js

If it works then all fine

Errrm, NOOOOO THIS IS WRONG and needs to be fixed!

because it does not need

False, this breaks on disconnect/reconnect, eg moving the element. Try to resize the page to get tabs to convert to accordion, then this is broken!

I thought it accessible enough

Well, it's not

@Hackwar sorry I cannot reproduce the error (I tried with a tags and an article).
Can you please try 2 thing:
1 update js, run npm install (if you did not made it before)
2 enable debug, and tell a full error trace (make screenshot or copy/paste here), it should show a correct line where error happened in choices.js instead of choices.min.js

@dgrammatiko

NOOOOO THIS IS WRONG and needs to be fixed!

but it is working ?

False, this breaks on disconnect/reconnect, eg moving the element.

It does not fail because lack of mutation observer, it fail because the tab CE move a content of sections around (I do not ask why ?), and so disconnectedCallback clean up the Choices instance.

TypeError: html.replace is not a function[Weitere Informationen] choices.js:5921:11
stripHTML
http://localhost/j4tags/media/vendor/choicesjs/js/choices.js:5921:11
addItemText
http://localhost/j4tags/media/vendor/choicesjs/js/choices.js:152:46
_canAddItem
http://localhost/j4tags/media/vendor/choicesjs/js/choices.js:1456:79
_handleChoiceAction
http://localhost/j4tags/media/vendor/choicesjs/js/choices.js:1395:27
_onMouseDown
http://localhost/j4tags/media/vendor/choicesjs/js/choices.js:2047:12
_onMouseDown self-hosted:974:17

This is a copy of the trace of the problem. Even after pulling the latest changes and doing a clean npm i, I still get this error.

All of these discussions about JS issues should however not prevent this PR from being merged.

@Hackwar by looking the trace, it seems that value not a string.
I have no idea why it happen in your case.
Can you try open a direct link for AJAX search for "green" in browser (if you have blog data, or for any other tag):
/index.php?option=com_tags&task=tags.searchAjax&like=green
in response you should get something like:

[{"value":"4","text":"Green","path":"green"},{"value":"5","text":"Green\/Lime","path":"green\/lime"}]

where "value":"4", is a string, and "value":4, would be an integer.

One thing I guessing, maybe you have different database version, and Joomla\CMS\Helper::searchTags returns an integer for you.

I have MySQL 5.7.24.

It not related to current pull request.

[{"value":2,"text":"test","path":"test"},{"value":3,"text":"test\/new","path":"test\/new"},{"value":4,"text":"test joomla","path":"test-joomla"}]
That is the response that I get. PHP 7.2.9, MariaDB 5.5.5

Thanks for checking. As I expected the type is not a string ("value":2, where 2 is integer)
MariaDB return ID as integer and MySQL as string, and the script expect a string.
I will check what can do with the script.

I have tested this item ✅ successfully on e66e9fd

I have tested this item successfully. The component behaves as expected from the users's poit of view.

For a test ist is necessary to add tags and therefore the patch #23395 mus be applied before testing.

I have tested this item ✅ successfully on e66e9fd

RTC

Thanks!