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

(fix) O3-2719: Inconsistent UI in visit summaries tabs when Empty State #1567

Merged
merged 1 commit into from
Jan 8, 2024

Conversation

psworld
Copy link
Contributor

@psworld psworld commented Jan 8, 2024

Requirements

  • My work includes tests or is validated by existing tests.

Summary

Inconsistent styles are present in Visit summaries tabs when Empty state is present. e.g Empty state for Notes tab is a simple text saying No notes found while for Tests tab it's a styled <EmptyState /> component.
In this PR I have changed all of the empty state of tabs to the <EmptyState /> component so that the style is consistent.
I have also modified tests accordingly

Screenshots

Before

image
image

After

image
image

Related Issue

Link to JIRA ticket: https://openmrs.atlassian.net/browse/O3-2719

@psworld
Copy link
Contributor Author

psworld commented Jan 8, 2024

@denniskigen Hello sir,
Kindly review the changes. If any more changes are required, please let me know, I will be more than happy to contribute and make necessary adjustments.

Thankyou for your time and consideration 🙏

Copy link
Member

@denniskigen denniskigen left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks, @psworld!

@denniskigen denniskigen merged commit a2d0fab into openmrs:main Jan 8, 2024
6 checks passed
@psworld
Copy link
Contributor Author

psworld commented Jan 8, 2024

LGTM. Thanks, @psworld!

Thankyou so much Sir 🙏🙏🙏❤️

brandones pushed a commit that referenced this pull request Jan 11, 2024
…te (#1567)

(fix) O3-2719: Inconsistent UI in visit summaries tabs

Co-authored-by: Ps <[email protected]>
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.

2 participants