Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add custom search-engine to query shiori #59

Merged
merged 1 commit into from
Oct 29, 2023

Conversation

patrickpichler
Copy link
Contributor

It is now possible to query Shiori directly from the firefox search bar by simply prefixing the query with @shiori. This will trigger a shiori keyword search. It is also possible to search by tags, by prefixing words with #.

fixes: #58

@fmartingr
Copy link
Member

Hey @patrickpichler thanks for the contribution! This seems super useful. One question before a proper review, how will browsers that don't support omnibox react to this? Will the execution break since browser.omnibox is not defined?

@patrickpichler
Copy link
Contributor Author

Great question, I have no idea tbh! Is the extension supported for Safari?

@fmartingr
Copy link
Member

Great question, I have no idea tbh! Is the extension supported for Safari?

I'd say is not officially supported for any browser, since it's mostly an alpha. Just trying to avoid crashes as much as possible. Maybe just adding an if (browser.omnibox) would do the trick here? Just to be sure.

@patrickpichler
Copy link
Contributor Author

Sounds good 👌 🙂
I will add it!

It is now possible to query Shiori directly from the firefox search bar
by simply prefixing the query with `@shiori`. This will trigger a shiori
keyword search. It is also possible to search by tags, by prefixing
words with `#`.

fixes: go-shiori#58
@patrickpichler
Copy link
Contributor Author

I wrapped it in an if :)

Copy link
Member

@fmartingr fmartingr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems a great addition, didn't knew about the omnibar exposed APIs. :) Thanks for your work!

@fmartingr fmartingr merged commit 137b2f3 into go-shiori:master Oct 29, 2023
1 check passed
@patrickpichler patrickpichler deleted the feature/add-searchengine branch November 3, 2023 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Shiori as custom search engine to quickly search through bookmarks
2 participants