Repository navigation
fix: keyboard-navigable dropdowns, double-submit guard, storage key h… - #815
Merged
nonsobethel0-dev merged 1 commit intoOct 17, 2026
Conversation
…ardening, offline indicator Keyboard navigation (a11y) - Fix CategoryFilter render (mismatched JSX tags from a bad merge) and add ArrowUp/ArrowDown alongside Left/Right/Home/End. - Add a reusable Listbox (WAI-ARIA select-only combobox) and use it for the policies sort selector: Arrow/Home/End, Enter/Space, Escape, typeahead. - Claims export menu: focus moves into the menu, Arrow/Home/End navigation, Escape returns focus to the trigger. Double-submit - Add useSubmitGuard: ref-based in-flight lock plus a 500ms leading-edge cooldown. Fixes double-clicking "Confirm details" in BuyPolicyModal also triggering signing. Applied to BuyPolicyModal and DepositModal, with a button spinner and aria-busy. Storage - Centralize every key in a storageKeys registry under the ps_ prefix and document them in docs/STORAGE.md; a test audits src for raw keys. - Move wallet id/address/network from localStorage to sessionStorage. - Fix useExpiryWarnings writing the raw wallet address into a key name. - migrateLegacyStorage() cleans up older builds' keys without logging active sessions out; clear the wallet kit's key on logout. Offline - networkStatus store combines navigator.onLine with API requests that get no response, probes the origin to auto-recover. - OfflineBanner in the layout; network actions (connect, buy/sign, deposit, withdraw) are disabled while offline. Also fix mismatched ErrorBoundaryWrapper tags in the root layout. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
@Enochflames Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
❌ Deploy Preview for boisterous-sunshine-dd4c4c failed.
|
nonsobethel0-dev
merged commit Oct 17, 2026
9f1aeaf
into
Parashield-Protocol:main
0 of 4 checks passed
DooseWayo
pushed a commit
to DooseWayo/parashield-frontend
that referenced
this pull request
Oct 21, 2026
…double-submit-storage-offline
rhemy-arc
pushed a commit
to rhemy-arc/parashield-frontend
that referenced
this pull request
Oct 21, 2026
…double-submit-storage-offline
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
closes #787
closes #788
closes #789
closes #790
…ardening, offline indicator
Keyboard navigation (a11y)
Double-submit
Storage
Offline
Also fix mismatched ErrorBoundaryWrapper tags in the root layout.