NavyaAilaS225272001/fix/messages-keyboard-shortcuts

[NavyaAila]

Added keyboard shortcuts for search, filter actions, and message deletion. Indicator is also there for available shortcuts.

[NavyaAila]

[Krisha190235]

Nice idea, and the shortcuts hint bar is a good touch. A few things to rework before merge, though:

The hook drives everything through the DOM — querySelectorAll('button') + textContent === 'Cancel'/'Delete' + .click(), and querySelector('input[placeholder*="Type a name"]'). That's fragile (breaks on any label/placeholder change or i18n) and this component already has the real handlers to call. Could we:

Use a ref on the search input instead of a placeholder query.
Call handleApplyFilters / handleClearFilters / openDeleteConfirm / closeDeleteConfirm directly rather than finding and clicking DOM buttons. (onDeleteSelected is currently passed in but never used — the Delete path scrapes the DOM instead.)
Reconsider the Delete shortcut — hasSelectedMessage just means "an active message exists," so Delete targets the first row arbitrarily, which is risky for a destructive action. I'd either drop it or wire it to a real selected row.
Esc likely duplicates the Modal's built-in close — worth confirming.

Happy to re-review once it's using refs/handlers instead of DOM scraping. A short clip of the shortcuts working would be great as evidence too.

[NavyaAila]

Thank you for the quick review, i have gone ahead and implemented the changes and included a clip showing the shortcuts are working in the message page. Could you please review it and approve the PR. Thank you!
PR503.zip

[NavyaAila]

CTRL + SHIFT + F to highlight the Sender Fields

CTRL + ENTER

Command activates the filter

CTRL + BACKSPACE

Filters cleared:

Delete confirmation:

Clicking ESC will remove the prompt and will not delete