fix: #2767 Search field loses focus after search

[Koc]

This PR closes #2767. Better to review with ignored whitespaces and in split view

nextcloud-forms-2025-08-19_21.39.03.mp4

[Koc]

Mates, I don't know how to fix eslint, please help 🙏

[Chartman123]

The lint error comes from a wrong filename in the import :)

[susnux]

I do not see how this fixes the issue - it just delays it

[Koc]

@susnux mate, I properly tested PR before submitting. To prove that fix works I've added a video to the PR description. There is only one case when we're loosing focus: by click of "clear" button. But I'm pretty sure that its normal behavior of ui library.

I'm using latest version of the Firefox on Ubuntu. But it was tested also on Chrome.

[Chartman123]

@susnux Could it be that the issue was caused by the order of elements? Especially the NcEmptyContent before the Header and the if/else branches within the template?

The debounce was there before, too.

@Koc yes, please stick with the debounce function and remove the custom utils/debounce.js

[Koc]

@Chartman123 @susnux guys, it not works as expected without this new debouncedProperty that I've added in current PR.

The main difference of this computed property comparing to the old implementation is that now we're delaying write to submissionSearch. Previously we write immediately but delay reading (watch).

Here an video which shown that we're still loosing focus when submissionSearch was cleared quickly right after there were no search results. At the same time debouncedProperty works fine for that scenario.

nextcloud-forms-2025-09-07_15.58.56.mp4

[Chartman123]

@Koc your latest commit is reverting changes to the icons. Please keep the changes to use the outlined icons :)

[Koc]

@Chartman123 good catch! It happened during conflict resolution. Now should be better, please check one more time

[Chartman123]

@Koc I think I found a solution for keeping our default "debounce" function and eliminating the focus loss. We have to switch the condition for disabling the PillMenu, the search <div> and the "No submissions" NcEmptyContent from noSubmissions to form.submissionCount === 0.

I changed noSubmissions to be based on the form.submissionCount and introduced a new noFilteredSubmissions that is based on this.submissions.length.

With that change we don't have to rely on the debouncing but fix the root cause of the focus loss by removing unneccessary re-renders.

I added a commit to your PR. Please review and test it 🙂

[Koc]

@Chartman123 It works! Thank you

[Chartman123]

@Koc great :) You can just squash the two commits into one and then we can merge the fix 👍🏻