TinyMCE insert contact popup: unwanted focus shift

[PeterNerlich]

Describe the Bug

When opening the popup to insert a contact, the focus is moved to to the submit button instead of staying on the search field. This is bad user experience, since the user likely wants to search for a different term next – submitting does not make sense and is not allowed by the system.

Steps to Reproduce

1. Go to a page or location
2. Click on Insert → Contact…
3. Type something in the input field
4. Change your mind and press backspace until the field is empty

Expected Behavior

The focus is still on the input field. The user can immediately type in another phrase to search for.

Actual Behavior

The focus is moved to the Submit button. The user probably doesn't notice and types in another phrase, only to find out they have to re-select the box and type it in again.

Additional Information

Should this prove difficult to solve properly with our loosely spit-glued together TinyMCE plugin + TomSelect, but are able to trigger a function the moment the focus shift happens, then as a workaround we might just check if we have a valid contact selection and if not, switch focus back to TomSelect

[charludo]

I can't reproduce this on FF. This:

1. Go to a page or location
2. Click on Insert → Contact…
3. Type something in the input field
4. Change your mind and press backspace until the field is empty

results in the expected behavior described above for me, only this:

1. Go to a page or location
2. Click on Insert → Contact…
3. Type something in the input field
4. press Enter
5. the input is moved to the submit button

changes the focus, but that's on purpose for the most common workflow.

Is this Chrome-only? If not, this is really curious, because it goes against the explicit logic already in the JS code 🙈

[juliankehne]

I also wasn't able to reproduce this. I tested it on FF + Opera (Windows) and also on Chrome (Ubuntu).