Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Findbar lags sometimes losing characters #3876

Closed
jkup opened this issue Sep 10, 2016 · 14 comments
Closed

Findbar lags sometimes losing characters #3876

jkup opened this issue Sep 10, 2016 · 14 comments

Comments

@jkup
Copy link
Contributor

jkup commented Sep 10, 2016

Did you search for similar issues before submitting this one?
Yes.
Describe the issue you encountered:
Sometimes when you open the findbar and start typing there is a noticeable lag before the characters show up. We think it might be blocking on finding all of the characters you typed and perhaps debouncing that query would fix it.

Expected behavior:

  • Platform (Win7, 8, 10? macOS? Linux
    macOs
  • Brave Version:
    0.12.0
  • Steps to reproduce:
    1.
    2.
    3.
  • Screenshot if needed:
  • Any related issues:
@BrendanEich BrendanEich added this to the 0.12.2dev milestone Sep 11, 2016
@Sh1d0w
Copy link

Sh1d0w commented Sep 16, 2016

This is possible related issue #3730

I think online suggestions are causing delay as well, but I haven't digged into the code to check that, decided worth mentioning it here :)

Those suggestions have to be async, not waiting for all of them to complete to show the suggestions list.

@bbondy bbondy modified the milestones: 0.12.3dev, 0.12.2dev Sep 17, 2016
@bbondy bbondy modified the milestones: 0.12.4dev, 0.12.3dev Sep 26, 2016
@bbondy
Copy link
Member

bbondy commented Sep 26, 2016

@BrendanEich or anyone else that can reproduce, please confirm that while you have it in a state of reproducing (make sure of this first!), can you turn off preferences -> search -> show tab matches. After you do that, does it reproduce?

@diracdeltas
Copy link
Member

@Sh1d0w @bbondy are your comments relating to the URL bar or the findbar? i don't think the findbar has suggestions.

@jkup is the user input lost, or does it just not show up for some delay?

@jkup
Copy link
Contributor Author

jkup commented Oct 5, 2016

@diracdeltas honestly I'm not sure I've encountered this. I know @BrendanEich has mentioned it a few times and asked me to create an issue.

@bbondy
Copy link
Member

bbondy commented Oct 5, 2016

Brendan has lag with both

@Sh1d0w
Copy link

Sh1d0w commented Oct 5, 2016

@diracdeltas Sorry I've thought of the URL bar, not the FIndbar. I have never had problems with the Findbar, rather than the URL bar (which have a lot of glitches tho, doesn't always autocomplete, text can't be "cut" sometimes etc).

Please ignore my comment in this thread regarding the FIndbar.

@diracdeltas
Copy link
Member

i definitely see it slower some times than others, but never enough to lose any characters. trying out a fix now.

diracdeltas referenced this issue Oct 5, 2016
This changes the findbar from a controlled React element to a
mostly-uncontrolled one, so the characters show up in the input
field before React state is propagated. When the frame is changed
or the findbar component is created for the first time, the input
is restored from saved state.

Auditors: @bbondy

Test Plan:
1. open some tabs
2. cmd+f to find a string in one of the tabs
3. switch to a different tab, cmd+f for a different string
4. switch back to the first tab - the original input should still be in the findbar
@diracdeltas
Copy link
Member

fixed by 27bd13f i think, please reopen if not

@bradleyrichter
Copy link
Contributor

I am still losing the "om" in ".com". I see this in both 12.6 and current master.

@bradleyrichter
Copy link
Contributor

further sleuthing has revealed an interesting sequence.

@diracdeltas - please DL this GIF and open it in mac preview. Then step through frames 76 - 84.

missingcharacters_in_url

@Sh1d0w
Copy link

Sh1d0w commented Oct 25, 2016

@bradleyrichter it seems it is still happening, but I can not reproduce it. Good you recorded it in a GIF. Now that I am looking into the frames one by one we can clearly see another issue I have reported some time ago #4219

Could be possibly related, or not.

@bradleyrichter
Copy link
Contributor

@Sh1d0w It does seem related.

@diracdeltas
Copy link
Member

diracdeltas commented Oct 26, 2016

@bradleyrichter i'm confused why you reported this here since your screenshot shows the urlbar, not the findbar. can this be closed?

if you're still seeing this in the latest master, please open a new issue.

@diracdeltas diracdeltas removed this from the 0.12.8dev milestone Oct 26, 2016
@bradleyrichter
Copy link
Contributor

I think a search for URL bar brought this up since it became mixed. I'll make a new issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

9 participants