Add Dash Extension

[RSO]

Description

Added the initial version of the Dash extension

Type of change

• New extension

Screenshot

CleanShot.2021-10-14.at.16.31.10.mp4

Checklist

• I read the documentation about publishing

[RSO]

I think that there might be a race condition in the search code, as we don't really cancel previous search attempts when typing quickly. We should be able to fix this using an abortController but I didn't feel like I was going to be able to get that done today, so wanted to publish this command as is.

[andreiborisov]

I there a way to directly search a certain docset without going in Find in Dash first?

[fturcheti]

@RSO I've sent some improvements in #126, but I don't know if that's the best way to contribute to this WIP. Let me know if there's a better workflow for this. And it would be great if you could comment my contribution too.

[pernielsentikaer]

Awesome @RSO 💪

It seems like it's almost ready to be shipped when you have reviewed and maybe adapted changes from #126

Also, could you resize the image to 512x512?

[PitNikola]

I also sent some feedback to @RSO in Slack in regards to this extension (disclosure: he's Raycast employee 😅), he's planning to take a look soon.

[RSO]

Yeah, sorry folks, flu-season hit me hard this week. I plan on merging in the fixes from @fturcheti next week!

[fturcheti]

Thanks, @RSO.

There's one thing left that still bothers me in the extension:

raycast-extension-dash-search-enter.mp4

After selecting a docset entry in Raycast and hitting "Open in Dash", the Dash window is opened with the query entered in its search input but you still have to hit enter to load the result. And this small quirk makes this integration less fluid and intuitive.

[fturcheti]

There's one thing left that still bothers me in the extension

After taking a look at #202, I managed to fix that here: 625aa56.

Although #202 has a better way of handling this specific action, I couldn't make it work with my Dash setup.

[yesmeck]

😅I submit a duplicate extension in #202, I should search open PRs first.

[fturcheti]

😅I submit a duplicate extension in #202, I should search open PRs first.

@yesmeck But it was due to your contribution that I could fix this quirk that was bugging me. Thanks! 🤝

[RSO]

CleanShot.2021-10-25.at.13.41.51.mp4

I got the multi-search working! You can now start typing just like in Dash itself and it'll search in all docsets. The only caveat is that if you do want to search inside a specific docset there is no clear identifier in the UI that distinguishes itself from the multi-docset search. Overall I think this is a massive UX improvement though, as you can now bind the command to a hotkey and just start typing!

[fturcheti]

I almost never make a global search in Dash. I have small keywords assigned to every docset, so I always use {keyword}:{query} when searching. But the way you implemented this seems to work well for both kind of uses.

I just installed this new implementation and I'll try it a little bit. I'm getting some errors when using it for now, but I couldn't find the cause yet. Is it working flawless for you?

[RSO]

I almost never make a global search in Dash. I have small keywords assigned to every docset, so I always use {keyword}:{query} when searching. But the way you implemented this seems to work well for both kind of uses.

Yeah, that was my thinking as well. In the future it'd be cool if we could dynamically register commands in the extension API so that you can set hotkeys for specific docsets yourself, but for now this works as well I guess 😉 .

I just installed this new implementation and I'll try it a little bit. I'm getting some errors when using it for now, but I couldn't find the cause yet. Is it working flawless for you?

The only issue I found was that when you type quickly there was a race-condition because we didn't cancel the old ./dashAlfredWorkflow call. I have fixed that in the latest version. Now it seems to work well for me, let me know if you still run into issues!

[RSO]

@fturcheti if you don't find any more issues after my latest changes I think that this PR is ready to get merged! Let me know what you find.

[fturcheti]

Hey @RSO! Yeah, your last commits fix the race condition (that was happening more often with the global search). For now, there's only this small bug left that I've found.

[RSO]

Awesome! Let's get this baby in the store!

[RSO]

Thanks for all the help @fturcheti @yesmeck @pernielsentikaer !