In order to do this, we added a keyword under the \u201comnibox\u201d property in the manifest file of our extension.<\/p>\n\n\n\n
\/\/ manifest.json\"omnibox\": {
\"keyword\": \"go\",
}<\/pre>\n\n\n\nThis omnibox go links search feature is a power feature that our users love and continue to use. The feature worked great, except in one particular use case. If you wanted to do a Google search for anything that started with \u201cgo<\/a>\u201d it would trigger the GoLinks<\/a> omnibox search, and prevent you from doing a Google search.<\/p>\n\n\n\nGo is a very common word to search if you write code in golang or if you search for other \u201cgo<\/a>\u201d related items.<\/p>\n\n\n\n\n- \u201cgo lang switch syntax\u201d<\/li>\n\n\n\n
- \u201cgo fund me\u201d<\/li>\n\n\n\n
- \u201cgo forward\u201d<\/li>\n\n\n\n
- \u201cGo Daddy\u201d<\/li>\n\n\n\n
- \u201cGo Blue!\u201d<\/li>\n<\/ul>\n\n\n\n
Each of these searches would trigger the GoLinks<\/a> omnibox search feature and frustrate our users from doing their work. So we had to seek out a solution.<\/p>\n\n\n\nOur solution was to change the \u201cgo<\/a>\u201d term to \u201cgo\/<\/a>\u201d to ensure that the omnibox functionality was triggered intentionally. It also followed the same syntax as a go link so it was a sound solution. This fix worked great in Chrome, however, it caused Firefox to throw an error and prevent the extension from loading.<\/p>\n\n\n\n![\"\"](\"https:\/\/images.golinks.io\/blog\/wp-content\/uploads\/2021\/02\/18190718\/image-6.png\")
<\/figure>\n\n\n\nIn Chrome, you are allowed to use special characters in the keyword that triggers the omnibox functionality, however in Firefox 62.0, adding a slash would throw this message: There was an error during installation: Extension is invalid<\/code>. So we filed a bug with Mozilla since the Web Extension standard should allow this behavior.<\/p>\n\n\n\n![\"\"](\"https:\/\/images.golinks.io\/blog\/wp-content\/uploads\/2021\/02\/18190720\/image-7-1024x317.png\")
https:\/\/bugzilla.mozilla.org\/show_bug.cgi?id=1500320<\/a><\/figcaption><\/figure>\n\n\n\nWe continued to receive customer complaints about users searching for \u201cgolang\u201d syntaxes or \u201cgo fund me\u201d campaigns, and they would have to uninstall the extension in order to search for \u201cgo\u201d related items. The bug was filed as a low priority (P3) <\/strong>by the Mozilla community, however, for us, this was a high priority issue.<\/p>\n\n\n\n![\"\"](\"https:\/\/images.golinks.io\/blog\/wp-content\/uploads\/2021\/02\/18190724\/image-8-1024x302.png\")
<\/figure>\n\n\n\nIt was also marked as a good-first-bug <\/strong>followed by a link on how to contribute if someone wanted to take on this bug. So, as any good engineering team would do, we planned a way to solve this issue for our customers directly: by fixing the bug in Firefox.<\/p>\n\n\n\n![\"\"](\"https:\/\/images.golinks.io\/blog\/wp-content\/uploads\/2021\/02\/18190730\/image-9-1024x187.png\")
<\/figure>\n\n\n\nWe realized that fixing the bug directly was a more proactive approach to solving the issue, than waiting around for someone else to fix the problem. Firefox is an open source browser, so anyone can go in and fix any issues in the backlog, including engineering teams like us. None of us had ever worked on writing code for a browser before, but when your customers have an issue, you have to do everything in your ability to solve their issue.<\/p>\n\n\n\n
On our team<\/a>, we use the common version control software: Git. However, to contribute to Firefox you must use a version control called Mercurial. Once the repository is downloaded you can run the build command to build the browser. This took a very long time to compile the first time, so we let it run overnight.<\/p>\n\n\n\nNext, we found the bug and made the necessary changes. We compiled the browser and tested to see if it worked manually. In addition to manual tests, we had to write unit tests for our change. Tests are important because it gives the reviewers confidence that the code change does what is expected, and it prevents unexpected code from breaking the current functionality.<\/p>\n\n\n\n
Once the code changes and tests were in, we submitted the changes to Phabricator which is the software used to review code.\u2699 D12242 Updating the omnibox keyword regex to allow trailing slashesPreviously the omnibox keyword would not allow trailing slashes, such as go\/ Chrome allows this keyword, and Firefox\u2026phabricator.services.mozilla.com<\/a><\/p>\n\n\n\nFrom here we did a normal code review with the community. The reviewer, Rob Wu, suggested that our Regular Expression could be further simplified.<\/p>\n\n\n\n![\"\"](\"https:\/\/images.golinks.io\/blog\/wp-content\/uploads\/2021\/02\/18190735\/image-10-1024x341.png\")
<\/figure>\n\n\n\nWith the necessary edits, our code was approved.<\/p>\n\n\n\n![\"\"](\"https:\/\/images.golinks.io\/blog\/wp-content\/uploads\/2021\/02\/18190738\/image-11-1024x293.png\")
<\/figure>\n\n\n\nNext, it needed to be accepted by a module peer before it landed.<\/p>\n\n\n\n
When the revision was ready to land it got merged into master and was ready to be released in a future version.<\/p>\n\n\n\n![\"\"](\"https:\/\/images.golinks.io\/blog\/wp-content\/uploads\/2021\/02\/18190742\/image-12-1024x257.png\")
<\/figure>\n\n\n\nFor us this was great, but there was also a small problem that prevented us from getting the new feature out to our users. The fix was now available in 65.0a1, which is the Alpha version of Firefox Nightly, but all of our users were using the latest publicly available version which, at the time, was 63.0.3.<\/p>\n\n\n\n
At this point, it was just a waiting game. We would need customer support to let our users know that a fix was on the way, but it was out of our control to release this feature.<\/p>\n\n\n\n
Eventually, version 65 was released to the public, and our fix could go out. We updated our extension with the changes, updated our users about the change, and responded to the happy customers that have been waiting for a fix for this feature.<\/p>\n\n\n\n![\"\"](\"https:\/\/images.golinks.io\/blog\/wp-content\/uploads\/2021\/02\/18190746\/image-13-1024x1006.png\")
Email to our customers<\/a><\/figcaption><\/figure>\n\n\n\nThis whole process took about 3\u20134 months to go from identifying the bug to releasing a fix to the feature available to the public, but we moved as fast as we could once we learned customers were having an issue.<\/p>\n\n\n\n
If you are a current user of GoLinks<\/a>, be sure to check out the new omnibox feature which is triggered with \u201cgo\/<\/a>\u201d in the address bar of Chrome and Firefox.<\/p>\n\n\n\nIf you aren\u2019t a user of\u00a0GoLinks<\/a>, sign up today at\u00a0https:\/\/www.golinks.io\/<\/a>, and learn how companies are improving productivity and communication within their organizations.<\/p>\n\n\n\n
![\"\"](\"https:\/\/images.golinks.io\/blog\/wp-content\/uploads\/2024\/09\/25102826\/golinks_blog_footer_cta_2x_2x-1.webp\")
<\/figure>\n<\/div>\n<\/div>\n\n\n\n