Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve Login Page UX: Add Loading Indicator and Error Handlingfix blank screen when loading #406

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Erichao233
Copy link

PR title

Improve Login Page UX: Add Loading Indicator and Error Handling


PR descripion

Fix issue#257
This PR enhances the user experience on the Login Page by addressing the issue of a blank screen during loading. It introduces a loading indicator and error handling to provide better feedback to users.

🔧 Changes Made

  1. Loading Indicator:

    • Added a loading spinner that displays immediately when the login page is accessed, providing users with immediate feedback.
  2. Error Handling:

    • Implemented error detection for the iframe. If loading fails, a message ("Loading failed, please try again.") is shown to inform users.
  3. Conditional Rendering:

    • Used component states to control the display of the loading indicator and error message based on the loading status of the iframe.

Testing Performed

  • Verified that the loading spinner appears during loading.
  • Confirmed the spinner disappears once the iframe content loads successfully.
  • Tested error scenarios to ensure the error message displays correctly.

📈 Impact

These enhancements improve user feedback during the login process, ensuring users are informed of the system's status and reducing confusion.

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