Giter Site home page Giter Site logo

Comments (17)

Tuhljin avatar Tuhljin commented on May 27, 2024 1

Awesome. Looking forward to the update. I'm happy to have found this extension. The ones I tried in the past only searched tabs within the current window which did me no good. (I might have... way too many tabs open across many windows.)

from tab-search.

reblws avatar reblws commented on May 27, 2024

Hmm the issue with the input not focusing on a click is actually an FF bug, another user pointed this out in #3.

The input should be automatically focusing if you hit the keyboard shortcuts. I used to have a timeout set so the input would focus properly, but I removed it in 0.3.0 because of a change in how I initialized the popup window. Since it's an issue again I'll reinstate the focus timeout so the input focuses properly. I'll push out an update tonight fixing this. (To be honest I don't really understand why the input doesn't focus if a sufficient timeout hasn't been set)

As for customizing the shortcut, I feel your pain. There's an ongoing issue in #2 regarding this. Long story short, we just have to wait for the Firefox devs to implement custom shortcuts for WebExtensions. In the meantime I'll switch the default shortcut to Ctrl+E since it doesn't seem like many people like the current shortcut (although I really like the current shortcut, the idea is to get your hands ready for typing).

from tab-search.

Tuhljin avatar Tuhljin commented on May 27, 2024

Ctrl+E is already a default shortcut, used by FF and other browsers to select the search bar.

from tab-search.

reblws avatar reblws commented on May 27, 2024

Hmm tried it out in my browser and it states here Ctrl+K and Ctrl+J focus the search bar: https://support.mozilla.org/en-US/kb/keyboard-shortcuts-perform-firefox-tasks-quickly

from tab-search.

reblws avatar reblws commented on May 27, 2024

Ctrl+Shift+F makes a lot more sense the more I think about it, thanks for the suggestion.

from tab-search.

reblws avatar reblws commented on May 27, 2024

Okay the focus timeout should do it, I'll close this for now but let me know if it still doesn't work. The update should be live in a few hrs or so.

By the way, could you let me know your browser version and OS (maybe computer specs too if that's alright)? This would help me figure out why this issue occurred in the first place, as I didn't spot this on the few machines I tested on. The only reason I can think right now of why you ran into this is that you have a very fast computer.

from tab-search.

Tuhljin avatar Tuhljin commented on May 27, 2024

I have the updated version (Ctrl+Shift+F now works, thanks for that) but the input still isn't receiving focus. I wonder if it's because I have so many tabs open.

There is an animation when the popup appears which can take a moment to complete and it's a little rough (opens smoothly at first but then suddenly snaps to a larger size). That might be normal, though. It takes less than half a second most of the time (but can take longer, generally if it's the first time or maybe if I haven't used it for a while).

I'm using Firefox 56.0.2 (64-bit) on Windows 10 with Intel i5 (4 cores, ~3.5GHz).

from tab-search.

reblws avatar reblws commented on May 27, 2024

How many tabs do you have open?

from tab-search.

reblws avatar reblws commented on May 27, 2024

Seems like this is most likely a browser bug as mentioned in #3: https://bugzilla.mozilla.org/show_bug.cgi?id=1324255

I don't know why I'm not affected on Linux though, it looks like it should affect everyone. I'll try to get this replicated on my end so I can test the setTimeout workaround more thoroughly.

from tab-search.

reblws avatar reblws commented on May 27, 2024

According to this reviewer they only have the issue with the input focus when the popup badge is in the Overflow menu.

@Tuhljin, @migo77: is your TabSearch icon in the overflow menu as well?

from tab-search.

migo77 avatar migo77 commented on May 27, 2024

For me, there's no difference! As I mentioned earlier, I noticed the same for LastPass so I believe it's a FF issue.

from tab-search.

ridhoq avatar ridhoq commented on May 27, 2024

+1 on this issue. I'm on Windows 10, Firefox Quantum. Ctrl-Shift-F opens the modal correctly, but the text input doesn't get focused. It doesn't matter how many tabs I have open. If the issue is upstream, is there any work around available until it gets fixed? Thanks!

from tab-search.

Tuhljin avatar Tuhljin commented on May 27, 2024

is your TabSearch icon in the overflow menu as well?

I've tried it both ways and it doesn't make a difference.

from tab-search.

reblws avatar reblws commented on May 27, 2024

Can someone try this build out? tabsearch-0.3.6-possible-input-fix.zip

from tab-search.

ridhoq avatar ridhoq commented on May 27, 2024

@reblws Looks like no dice. I had to grab Firefox Developer edition and install your fix but looks like the same behavior as before. If you want to add some logging to that build, I can run it and screenshot the output. Also open to other suggestions.

from tab-search.

reblws avatar reblws commented on May 27, 2024

@ridhoq Thanks for trying. That was the only workaround I know of. The only other thing I can think of is just gradually increasing the timeout values in https://github.com/reblws/tab-search/blob/dev/fix-inputfocus/src/core/pages/popup/side-effects.js#L69 . If anyone has any suggestions let me know.

On the bright side it looks like Zilla's making progress on a patch for this popup focus bug.

I'm going to put the fix in for 0.3.7 since it doesn't really break anything, maybe it might help a few other people.

from tab-search.

reblws avatar reblws commented on May 27, 2024

Good news, looks like it's fixed in Firefox 60

from tab-search.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.