Safari: Dropdown for multiselects opens immediately#1385
Safari: Dropdown for multiselects opens immediately#1385Arnei wants to merge 1 commit intoopencast:r/17.xfrom
Conversation
In Safari, open the details of an event and click on one of the multi-select fields (e.g. presenters, contributors). This will open up a dropdown which hides already selected values. This is not desirable. Like in other browsers, the dropdown should only open after clicking the field an additional time. This patch fixes the issue by removing autoFocus attribute from the input element. This has an annoying side effect in that while you now need two clicks in Safari, you need three clicks in other browsers (e.g. Firefox). Arguably a better solution would be to get rid of `<datalist>` entirely, as it is not well supported. But this quick fix should be acceptable for now.
|
Use Run test server using develop.opencast.org as backend: Specify a different backend like stable.opencast.org: It may take a few seconds for the interface to spin up. |
|
I don't have Safari, but I do have Chrome and Firefox, neither of which seem to be behaving quite right. Chrome: Screencast.from.2025-09-03.15-07-23.mp4Firefox: Screencast.from.2025-09-03.15-08-59.mp4In neither case does the UI look quite right to me - I would expect a React style dropdown the way that the License field works. I guess I don't care if it looks this way, but it's out of place when compared to the rest of the UI. |
|
Ah heck, thanks for testing. Then this PR just might not be the right approach and we'll have to get rid of |
|
Given that 17.x is pretty much EOL at this point, should this get either punted forward to 18, or refiled when it's ready? |
|
I'd say refile it. I'll open an issue so that the problem does not get lost. |
In Safari, open the details of an event and click on one of the multi-select fields (e.g. presenters, contributors). This will open up a dropdown which hides already selected values. This is not desirable. Like in other browsers, the dropdown should only open after clicking the field an additional time.
This patch fixes the issue by removing autoFocus attribute from the input element. This has an annoying side effect in that while you now need two clicks in Safari, you need three clicks in other browsers (e.g. Firefox). Arguably a better solution would be to get rid of
<datalist>entirely, as it is not well supported. But this quick fix should be acceptable for now.How to test this
Can be tested as usual. Test with different browsers, Safari and otherwise.