If the user initiates a search manually (doesn't matter where from), I think they'd expect to see something, even if the results are empty. I'm not entirely clear if this PR takes that into account or not.

But anyway, the real fix for #8626 would be to finish #7791 . @droidmonkey , how about including that in 2.7.4?

The code introduced here only ends the search if no entries are found when the "next search label" variable is set. This variable is only set when performing the expired entries on unlock search. Although it looks like the gui tests are failing on search.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 64.42%. Comparing base (37baa6f) to head (26973fe).
Report is 343 commits behind head on develop.

@@             Coverage Diff             @@
##           develop    #8643      +/-   ##
===========================================
+ Coverage    64.41%   64.42%   +0.01%     
===========================================
  Files          341      341              
  Lines        44242    44245       +3     
===========================================
+ Hits         28496    28504       +8     
+ Misses       15746    15741       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests
• JS Bundle Analysis - Avoid shipping oversized bundles