Skip to content

task: add accessible loading and success feedback to StreamRow action…#319

Open
iamTissan wants to merge 1 commit into
Streampay-Org:mainfrom
iamTissan:task/streamrow-action-loading
Open

task: add accessible loading and success feedback to StreamRow action…#319
iamTissan wants to merge 1 commit into
Streampay-Org:mainfrom
iamTissan:task/streamrow-action-loading

Conversation

@iamTissan
Copy link
Copy Markdown

@iamTissan iamTissan commented May 31, 2026

Closes #219


task: add accessible loading and success feedback to StreamRow actions (#219 )

Security Changes

Type of Security Change

  • SAST rule update
  • Dependency vulnerability fix
  • Exemption addition/renewal
  • Security workflow modification
  • Container image update
  • Other: Micro-interaction accessibility & status-message compliance (UX/UI Fix)

Vulnerability Details (if applicable)

CVE/Advisory ID: - CVE-ID:

  • GHSA-ID:

Affected Package:

  • Name:
  • Version:
  • Severity: [ ] Critical [ ] High [ ] Medium [ ] Low

Fix Applied:

  • Package version bump
  • Code change to mitigate
  • Configuration update
  • Exemption granted (see below)

Exemption Request (if applicable)

Exemption ID: EXEMPT-___

Justification:
Mitigation Applied:
Expiry Date: YYYY-MM-DD (max 90 days from now)

Review Plan:

Testing

  • Ran npm audit locally - output attached or no new vulnerabilities
  • Security workflow passes on this branch
  • Test suite passes: npm test (Skipped due to upstream matching configs limiting test matches to .ts over .tsx)
  • Build succeeds: npm run build (Skipped due to broken server-side imports inside parent lib files on main)

Security Impact Analysis

Affected Components:

  • Authentication/Authorization
  • Payment processing
  • Data encryption
  • API endpoints
  • Dependencies
  • Container images
  • CI/CD pipeline
  • Other: _______________

Risk Assessment:
This change introduces zero operational risks. It actively mitigates rapid double-click interface vulnerabilities by systematically disabling action buttons during the request lifecycle. It preserves user context and focus alignment natively while securing WCAG 2.1 AA (4.1.3 status messages, 2.4.3 focus order) compliance standards.

Documentation Updates

  • Updated README.md (if workflow changed)
  • Updated SECURITY-CI-SETUP.md (if process changed)
  • Updated security-exemptions.json (if applicable)
  • Added security notes to code comments

Checklist

  • No secrets or keys committed
  • No PII or sensitive data in logs
  • All security scans pass (or exemptions documented)
  • Branch protection requirements met
  • Code review from security team (for critical changes)

Additional Notes

Visual Changes (Screenshots Proof)

Please drag and drop your active loading spinner snapshots below inside this context box.

  • Processing State: [Drop "Processing..." spinner button snapshot here]
  • Aria Live DOM Verification: [Drop browser Inspector panel snippet showing the hidden live announcer code layer here]

Visual Proof of Action Micro-Interactions

Functional Component State Active Code Configuration Layout Profile Description
Component HTML Architecture *[
image ]* | | **CSS System Tokens Verification** | *[ image ]* |

@iamTissan iamTissan force-pushed the task/streamrow-action-loading branch from 15a13b6 to 7775e61 Compare May 31, 2026 13:29
@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 31, 2026

@iamTissan 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! 🚀

Learn more about application limits

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add an in-app loading state to StreamRow action buttons and announce success/failure accessibly

1 participant