eb987460f2
* DEV: Make emoji elements focusable Since emoji elements are of type `<img>` it requires a `tablindex="0"` in order to be focusable. * WIP: Handle emoji focus/selection via arrow keys Near completion, however, need a few fixes/improvements and overall code cleanup * WIP: Testing * DEV: Fixes and cleanup * DEV: Follow conventions * DEV: Improve up/down traversal when recents present * DEV: Emoji markup in tests should include `tabindex` * DEV: Add `tabindex` to topic tests * DEV: Variable name as `searchInput` instead of `searchBar` * DEV: Use appropriate method name (`_setNumEmojiPerRow`) * DEV: Add comments and avoid nested if * WIP: Adding test * Fix first test * DEV: Add assertions for arrow keys and escape key * Some fixes for up/down navigation This does not fix everything, when going from one section to another, there are issues * Fix a small regression * FIX: Ability to focus on search results Fixes regression * Refactor calculating next up/down emoji * Debugging test failure * Skip stubborn CI test, add others Co-authored-by: Penar Musaraj <pmusaraj@gmail.com> |
||
|---|---|---|
| .. | ||
| badge-test.js | ||
| bookmark-test.js | ||
| category-test.js | ||
| composer-test.js | ||
| email-log-test.js | ||
| group-test.js | ||
| invite-test.js | ||
| nav-item-test.js | ||
| pending-post-test.js | ||
| post-stream-test.js | ||
| post-test.js | ||
| private-message-topic-tracking-state-test.js | ||
| report-test.js | ||
| rest-model-test.js | ||
| result-set-test.js | ||
| session-test.js | ||
| site-test.js | ||
| staff-action-log-test.js | ||
| theme-test.js | ||
| topic-details-test.js | ||
| topic-test.js | ||
| topic-tracking-state-test.js | ||
| user-action-test.js | ||
| user-badge-test.js | ||
| user-drafts-test.js | ||
| user-stream-test.js | ||
| user-test.js | ||
| wizard-field-test.js | ||