cliffordfajardo / cato

Spotlight and alfred app like search in your browser 🔍.
https://chrome.google.com/webstore/detail/cato/icphdcfpompgbdikholnedfeidemgobg
MIT License
262 stars 22 forks source link

Can't change tab [Windows] #2

Open upgradeQ opened 7 years ago

upgradeQ commented 7 years ago

When I press Enter nothing happens.

cliffordfajardo commented 7 years ago

@HQupgradeHQ - hmmm. Are you using Windows/Mac?

I'm having trouble reproducing the error. So when you launch Cato, select the "Change Tab" command, no open tabs are showing up like in the GIF?

change-tab

upgradeQ commented 7 years ago

@cliffordfajardo I am using Windows. I can see tabs, but I can't change them. On this GIF I'm pressing: hotkey to Cato, change command, arrows, Enter

GIF

cliffordfajardo commented 7 years ago

I'll try to setup a Windows environment and fix this. Thanks for reporting this!

Some follow up questions:

upgradeQ commented 7 years ago

version : Version 61.0.3163.100 (Official Build) (64-bit) 'Find Bookmark' works .

Olian04 commented 7 years ago

@cliffordfajardo I have a windows environment i could try this on. I'l get to it later today.

cliffordfajardo commented 7 years ago

@Olian04 - in this branch I tried to use chrome.tabs instead of browser to see if the web-extension polyfil was the problem, but it doesn't look like it is. Even when using chrome. instead of browser. I'm having the same issues.

I also tried writing this in callback style and that didn't fix the issue.

A temporary fix I think (needs testing) is to just grab all the tabs for the current window ONLY, as opposed to all browser windows for the Windows operating system. The original code should stay the same for Linux/Mac.

Determining OS:

I think we'll need to do something like

//use destructuring to extract the os object.
const {os} = await browser.runtime.getPlatformInfo()
 if(os === 'win') {
   //grab all the tabs for the current window
} else {
 //use the original code
}

I'll look at this probably on Friday.

@Olian04 - just wanted to thank you for taking your time to learn about Cato and it's inner workings! Your the first officla contributor with your PR.

Hope I'm not being to nit-picky on the details for you 💪

Cheers

abass commented 7 years ago

Just came here to report this. Excellent work @cliffordfajardo - looking forward to a fix (:

upgradeQ commented 7 years ago

@cliffordfajardo Have a look on this extension, it's open source and has the ability to change window on windows.

sendsent commented 6 years ago

Hi @cliffordfajardo, just came here to report this too for Mac. When I type in "change to youtube" and have a youtube link up in another tab, it does not go there. No tabs show like in the GIF. Find Bookmark also has the same issue. Nothing will show.

Thanks - a :)

cliffordfajardo commented 6 years ago

@sendsent - I’m curious, what version of Google?

Thanks for reporting!

sendsent commented 6 years ago

@cliffordfajardo - Version 63.0.3239.132 (Official Build) (64-bit)

taylorjdawson commented 6 years ago

Has a fix for this been implemented?

ln3333 commented 5 years ago

Same thing for me, but what makes it more strange is that when I opened a console(inspect the extension element) and tried to see if there're some errors, it functions well!

OS: Mac Mojava 10.14 Chrome: Version 72.0.3626.96 (Official Build) (64-bit)