The quick filter should indicate in the thread pane that a search is still in progress.
Categories
(Thunderbird :: Search, defect, P3)
Tracking
(thunderbird_esr115 wontfix, thunderbird_esr128? affected)
People
(Reporter: mathias, Assigned: vineet)
References
(Blocks 1 open bug, Regression)
Details
(Keywords: regression, ux-userfeedback)
Attachments
(2 files)
619.25 KB,
video/mp4
|
Details | |
48 bytes,
text/x-phabricator-request
|
wsmwk
:
approval-comm-beta+
|
Details | Review |
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36
Steps to reproduce:
I entered a search term in 'Quick filter'(subject only) which resulted in few results.
I then clicked the 'Body' scope selection to find more results.
Actual results:
TB displayed all of the current folder as 'results' in the message index pane while processing the new search, (which is confusing [ a lie])
and only when that was finished displayed the actual results.
In fact, apart from a very small indication on the bottom, there is no indication that the search is stil ongoing and what is shown are not the result (neither the ones from before, nor the final ones, but instead it shows everything in the folder being searched!)
Expected results:
While re-searching my search-term with scope set to 'body',
See the attached MP4 (blurred for privacy). The initial 'subject or sender' search found only a single result. After switching to include 'Body', (which takes 8 seconds, [ from ca 0:00:05 to second 0:00:13 in the video] , the result window shows many more entries which are in fact not search results, ultil the search completes.
It would be useful for TB to
- not pretend it had found many more results while still searching
AND either
a) keep the old search result until the the message index pane. (not ideal)
b) somehow mark in the message index pane that what is shown is not final and that the search is still going on
I often have folders which are much larger and where a full body search can take many more seconds, even minutes . There being no indication that the search is still ongoing, and the message index pane showing many non-results during that time, it s hard to tell when it is ready or whether the result is in fact complete.
for option b) above, it might be useful to either
- colour the background if the message index pane while a search in progress
- blur that window
- or otherwise overlay a banner that indicates the search is not completed. (a progress meter sounds nice but would likely be lying as to expected completion). Maybe the banner should indicate time taken so far during the search?)
This is TB 116.6 on OSX
Updated•6 months ago
|
Updated•6 months ago
|
Comment 2•5 months ago
|
||
Needinfo Vineet on this so he can investigate the performance implications of reimplementing setTabThinking().
We might fallback to a different solution as we disabled that due to CPU spikes and unnecessary DOM layout reflows.
https://searchfox.org/comm-central/search?q=setTabThinking&path=&case=false®exp=false
Updated•5 months ago
|
A further aspect of this bug is that sometimes I will get the in progress list and one of them is the one I am looking for, so all good. But when I click on it, the message changes to another message that is also part of the search set but hasn't been displayed yet, so it makes this in progress display especially useless.
(In reply to jdien07 from comment #3)
A further aspect of this bug is that sometimes I will get the in progress list and one of them is the one I am looking for, so all good. But when I click on it, the message changes to another message that is also part of the search set but hasn't been displayed yet, so it makes this in progress display especially useless.
and since it's not giving me any indication of when the search is complete, I can't just wait till it is done.
TB 115.11.0 on Pop!_OS 22.04
Assignee | ||
Comment 5•2 months ago
|
||
I have not noticed any discernible difference in CPU spike when using the QFB for search with setTabThinking() re-instated. However, the setTabThinking() function does not behave consistently. It might be more appropriate to re-factor/re-implement this function with specific requirements for it's behaviour.
Assignee | ||
Comment 6•2 months ago
|
||
I'm going to tackle this bug by adding a small spinner where it shows the number of results found before they are found - eg while the search is happening. That way, we know the search is complete when the spinner is no longer there.
Assignee | ||
Comment 7•2 months ago
|
||
Updated•2 months ago
|
Comment 8•2 months ago
|
||
As noted in bug 1831500, setTabThinking()
is meant to be called to show "Searching..." in the status bar.
https://searchfox.org/comm-central/rev/b675b13ad0a6a34a682dcf29e54532e29b43db8b/mail/base/content/tabmail.js#1786
That's how it worked in 102. If you don't intend to restore that, you can remove the obsolete function.
Updated•2 months ago
|
Assignee | ||
Updated•2 months ago
|
Comment 9•2 months ago
|
||
We're still in the 129 cycle until July 8th
Comment 11•2 months ago
|
||
Pushed by kaie@kuix.de:
https://hg.mozilla.org/comm-central/rev/4300976cf55b
Add spinning svg to quick filter bar to show while search is progress. r=#thunderbird-front-end-reviewers
Assignee | ||
Comment 12•2 months ago
|
||
Comment on attachment 9411172 [details]
Bug 1874419 - Add spinning svg to quick filter bar to show while search is progress. r=#thunderbird-front-end-reviewers
[Approval Request Comment]
Regression caused by (bug #): -
User impact if declined: There would be no searching indication in the quick filter bar.
Testing completed (on c-c, etc.): Tested on cc, added test to ensure visibility of throbber.
Risk to taking this patch (and alternatives if risky): None.
Comment 13•2 months ago
|
||
Comment on attachment 9411172 [details]
Bug 1874419 - Add spinning svg to quick filter bar to show while search is progress. r=#thunderbird-front-end-reviewers
[Triage Comment]
Approved for beta
Comment 14•1 month ago
|
||
This is now in beta. Vineet, do we want to uplift this to 128?
Assignee | ||
Comment 15•1 month ago
|
||
Comment on attachment 9411172 [details]
Bug 1874419 - Add spinning svg to quick filter bar to show while search is progress. r=#thunderbird-front-end-reviewers
[Approval Request Comment]
Regression caused by (bug #): -
User impact if declined:There would be no searching indication in the quick filter bar.
Testing completed (on c-c, etc.): Tested on cc and beta, added test to ensure visibility of throbber.
Risk to taking this patch (and alternatives if risky): None
Assignee | ||
Comment 16•1 month ago
|
||
Comment on attachment 9411172 [details]
Bug 1874419 - Add spinning svg to quick filter bar to show while search is progress. r=#thunderbird-front-end-reviewers
[Approval Request Comment]
Regression caused by (bug #): -
User impact if declined: There would be no searching indication in the quick filter bar.
Testing completed (on c-c, etc.): Tested on cc, beta, added test to ensure visibility of throbber.
Risk to taking this patch (and alternatives if risky): None.
Comment 17•1 month ago
|
||
Comment on attachment 9411172 [details]
Bug 1874419 - Add spinning svg to quick filter bar to show while search is progress. r=#thunderbird-front-end-reviewers
[Triage Comment]
Approved for esr128
Comment 18•1 month ago
|
||
bugherder uplift |
Thunderbird 128.1.0esr:
https://hg.mozilla.org/releases/comm-esr128/rev/ade5082d4e97
Comment 19•1 month ago
|
||
Causing a test failure and possibly a crash on c-esr128:
PROCESS-CRASH | Missing chrome or resource URLs: chrome://messenger/skin/icons/spinning.svg [@ mozilla::net::CheckForBrokenChromeURL] | comm/mail/test/browser/quick-filter-bar/browser_filterLogic.js
TEST-UNEXPECTED-FAIL | comm/mail/test/static/browser_parsable_css.js | missing chrome://messenger/skin/icons/spinning.svg referenced from chrome://messenger/skin/shared/quickFilterBar.css -
spinning.svg was added in Bug 1905839 (tb129) and is not uplifted to c-esr128.
Comment 20•1 month ago
|
||
backout bugherder uplift |
Backout esr128:
https://hg.mozilla.org/releases/comm-esr128/rev/c5fd52b41670
Updated•1 month ago
|
Comment 21•26 days ago
|
||
If this is wanted on esr128, something needs to be done about comment 19.
Comment 22•14 days ago
|
||
I replied with what we should do about it in Bug 1905839, copying it here for easy tracking:
we should create a 128 dedicated patch that uses the m-c spinner
Comment 23•13 days ago
|
||
Comment on attachment 9411172 [details]
Bug 1874419 - Add spinning svg to quick filter bar to show while search is progress. r=#thunderbird-front-end-reviewers
Removing approval flag for now. Please re-request when the spinner patch for esr128 is ready.
Assignee | ||
Updated•5 days ago
|
Assignee | ||
Comment 24•5 days ago
|
||
Will create a bug for adding the m-c spinner and re-request when that is ready for esr128.
Description
•