mirror of
https://gh.wpcy.net/https://github.com/discourse/discourse.git
synced 2026-05-10 05:28:09 +08:00
d1f6e5f4f3
Some checks are pending
Licenses / run (push) Waiting to run
Linting / run (push) Waiting to run
Publish Assets / publish-assets (push) Waiting to run
Tests / core backend (push) Waiting to run
Tests / plugins backend (push) Waiting to run
Tests / core frontend (Chrome) (push) Waiting to run
Tests / plugins frontend (push) Waiting to run
Tests / themes frontend (push) Waiting to run
Tests / core system (push) Waiting to run
Tests / plugins system (push) Waiting to run
Tests / themes system (push) Waiting to run
Tests / core frontend (Firefox ESR) (push) Waiting to run
Tests / core frontend (Firefox Evergreen) (push) Waiting to run
Tests / chat system (push) Waiting to run
Tests / merge (push) Blocked by required conditions
## 🔍 Overview Fixes accessibility issue where <kbd>Ctrl</kbd>/<kbd>Cmd</kbd>+<kbd>Enter</kbd> on keyboard-navigated search results opened in the same tab instead of a new tab. Users can now use standard browser shortcuts when navigating search results with arrow keys. ## ➕ More Details Discourse prevents default browser behavior on Enter key events in search results to handle custom routing, analytics logging, and menu management. This custom handling needed explicit modifier key detection since the default browser <kbd>Ctrl</kbd>/<kbd>Cmd</kbd>+<kbd>Enter</kbd> behavior was bypassed. The fix adds proper <kbd>Ctrl</kbd>/<kbd>Cmd</kbd>+<kbd>Enter</kbd> handling while preserving all existing functionality. ## 📷 Screen Recordings ### ← Before https://github.com/user-attachments/assets/3b8715bf-c5bb-4f23-87f0-ee66b088b220 ### → After https://github.com/user-attachments/assets/e235a311-bf27-467c-93d5-a9233228fae3 ### 🔗 References https://meta.discourse.org/t/open-in-new-tab-browser-keybind-fails-on-search-results/384004?u=keegan |
||
|---|---|---|
| .. | ||
| app | ||
| config | ||
| lib | ||
| public/assets/scripts | ||
| scripts | ||
| tests | ||
| .ember-cli | ||
| .npmrc | ||
| .watchmanconfig | ||
| ember-cli-build.js | ||
| package.json | ||
| testem.js | ||
