Skip to content

fix : replaced alert with inline error in Analytics.jsx image export - #2054

Open
tmdeveloper007 wants to merge 1 commit into
aryandas2911:mainfrom
tmdeveloper007:#2049
Open

fix : replaced alert with inline error in Analytics.jsx image export#2054
tmdeveloper007 wants to merge 1 commit into
aryandas2911:mainfrom
tmdeveloper007:#2049

Conversation

@tmdeveloper007

Copy link
Copy Markdown

Summary of What Has Been Done

Replaced the browser alert() call in the Analytics page image export handler with a state-based inline error message for better UX.

Changes Made

  • Added exportError state to the Analytics component
  • Cleared exportError at the start of exportToImage
  • Replaced alert() with setExportError() on catch
  • Added inline error display below the export buttons

Impact it Made

  • Eliminates jarring browser dialog on export failure
  • Works properly on mobile
  • Consistent with the app's UX patterns

Note: Please assign this PR to the tmdeveloper007 account.

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.

1 participant