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

Improved the loading state UI in the Form Renderer Component #238

Closed
wants to merge 4 commits into from

Conversation

malak-elbanna
Copy link

Changes Made

Animated Loading Indicator

  • Replaced the InlineLoading component with the Loading component from the Carbon Design System.
  • The new loading indicator is animated and provides a visual to the user during the loading process.

User Feedback

  • Added a text message below the loading indicator to inform the user that the form is currently loading.
  • The message is localized support multiple languages using the useTranslation hook.

No Overlay

  • Configured the Loading component to display without an overlay.

Conclusion

The updated loading state in the FormRenderer component aims to create a more pleasant and engaging user experience during form loading times. By providing an animated loading indicator and a descriptive message, users are kept informed and assured that the application is actively working to load the necessary resources.

Related Issue

#237

Copy link
Contributor

@CynthiaKamau CynthiaKamau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @malak-elbanna , kindly address the conflicts

@malak-elbanna
Copy link
Author

Thanks @malak-elbanna , kindly address the conflicts

Resolved! Thank you

@hadijahkyampeire
Copy link
Contributor

@malak-elbanna there are new conflicts, please resolve them as well.

@NethmiRodrigo
Copy link
Contributor

Closing this as it has been inactive. Please feel free to re-open if you continue working on it. Thanks for the contribution!

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.

4 participants