Closed Bug 595533 Opened 9 years ago Closed 9 years ago
Characters are sometimes ignored when typing a filter search very fast
If you open Panorama and type two or more characters very quickly, the second typed character is sometimes ignored. So if you type "abc", you may end up with a search for "ac". I'm seeing this on Windows and I couldn't reproduce in my Linux vm.
My guess is that this is because the input box hasn't received focus yet.
Priority: P2 → P3
Target Milestone: --- → Firefox 4.0
This is a data-loss issue (although not a very bad one). I'm bumping it up to beta 8 because I think it is a quick win.
Assigning to Anant, assuming Sean hasn't started work on it.
Assignee: seanedunn → anant
This works for me on my windows 7 with latest nightly build. Sylvain: do you still see the issue?
No, works for me too now (Windows 7). Can anyone test on Linux?
I spoke too soon. I can still reproduce the issue on my laptop (Windows 7 too). There might be something hardware/driver related that makes it reproducible or not.
I am unable to reproduce the issue either on Windows 7 or Mac OS X. I tried typing as fast as I could! Likely to be a driver / hardware specific issue.
Yes, it has to do with the CPU speed of your machine. Grab a crappy netbook and you should be able to repro it no problem.
Another way to reproduce (even if you have a fast machine) is to open a lot of tabs. With 50 open tabs I can see the issue easily.
Note that the patch for bug 597399 changes from using keydown to using keypress... perhaps this will allow us to get rid of the setTimeout in ensureSearchShown, which is probably what's causing this bug.
I can confirm that the setTimeout in ensureSearchShown is causing the issue. Do you remember why it's needed? I tried to remove it and it seems to still work fine on Windows and Linux. The attached extension patches ensureSearchShown to remove the setTimout.
Moving over to Raymond
Assignee: anant → raymond
Moving to b9
Comment on attachment 496739 [details] [diff] [review] v1 Raymond, the patch looks fine, but it's going to conflict with bug 610821 (specifically "Patch for tabview v1.0"); let's hold off on this until that one lands.
Attachment #496739 - Flags: review?(ian) → review-
(In reply to comment #17) > Comment on attachment 496739 [details] [diff] [review] > v1 > > Raymond, the patch looks fine, but it's going to conflict with bug 610821 > (specifically "Patch for tabview v1.0"); let's hold off on this until that one > lands. It looks like the "Patch for tabview v1.0" would also fix this because it removes the setTimeout() in the ensureSearchShown().
bugspam (moving b9 to b10)
bugspam (removing b9)
No longer blocks: 598154
Bug 610821 also fixes this.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
verified with nightly build of minefield.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.