-
Notifications
You must be signed in to change notification settings - Fork 17
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
enter key input listener is too global #140
Comments
Hey @fritzmg
I did the following steps (using Chrome, Firefox and Edge): 1.
2.
In both cases, page was loaded correctly. |
For me it is hard to distinguish between Web Components and the Extension itself. All I can see is, that this behavior occurs in the Magento Webshop. If it's a problem within the FACT-Finder Web Components, then it needs to be fixed there of course. |
Firefox 68.0
Bitwarden, Passbolt, uBlock Origin, Xdebug
<ff-communication sid="…" user-id="…" url="…" version="7.3" default-query="*" channel="…" use-url-parameter="true" only-search-params="false" disable-single-hit-redirect="true" use-asn="true" use-found-words="false" use-campaigns="true" generate-advisor-tree="false" use-personalization="true" use-semantic-enhancer="true" use-aso="true" use-browser-history="true" use-cache="true" disable-cache="false" currency-code="GBP" currency-country-code="de-DE" /> |
I have found a reproduction now:
This triggers a search. |
Expected behavior
When entering an address in the address bar of the browser and hitting 'enter', I expect that address to be opened by the browser. When hitting Ctrl+Shift+Del (to delete the browser cache) and then hit 'enter', I expect no reaction from the web application.
Actual behavior
Whenever you hit the enter key, a global key listener will either execute a search or a redirect. So when you are already in the shop, but not on the FACT-Finder/result page and you enter a new address in the address bar of the browser - the browser will actually redirect you to the FACT-Finder/result page upon hitting enter, instead of opening the newly entered address.
Similarly, if you clear your browser's cache with Ctrl+Shift+Del and then Enter, FACT-Finder reacts to that as well.
Steps to reproduce
The browser will open the FACT-Finder/result page instead of the newly entered address.
It's very irritating ;)
The text was updated successfully, but these errors were encountered: