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

Bugs Detected After Merging PR #8737 , PR #8706, PR #8680 #8740

Open
11 of 12 tasks
shivankacker opened this issue Oct 8, 2024 · 6 comments · May be fixed by #8784
Open
11 of 12 tasks

Bugs Detected After Merging PR #8737 , PR #8706, PR #8680 #8740

shivankacker opened this issue Oct 8, 2024 · 6 comments · May be fixed by #8784
Assignees

Comments

@shivankacker
Copy link
Member

shivankacker commented Oct 8, 2024

Follow-up : #8737 , #8706 , #8680

Bugs

  • typo's are missing in the patient details page

Image

  • when you try to use restore button in the log update page, it won't restore any data to the form

Image

  • Sign out button icon is misaligned

Image

  • Make sure the minimum height for the content area takes up the height of the viewport across the platform

Image

  • The relative time for the "Restore" button on the form is calculated based on when the user entered and exited the form. For example, if the user accessed the registration form at 1:00 PM and reloaded the page at 1:05 PM, the relative time displayed on the button should indicate "a few seconds ago." However, it currently shows "5 minutes ago" because the relative time is being calculated from the moment the user first entered the form, rather than from when the page was reloaded.

Image

  • The breadcrumb navigation isn't functioning as expected, preventing users from returning to the immediate previous page. For instance, when a user is on the consent record page (which is part of the consultation page), clicking on the "admitted on..." breadcrumb should ideally take them back to the patient consultation page, but the now its not even clickable. This similar issue is occurring across the platform, where navigating back to the most recent previous page via breadcrumbs is consistently broken

Image

  • The Favicon is stretched. Use a 1:1 Favicon ratio

Enhancements

  • the width of the autocomplete text box and add button in Slideover such as linked facility, and linked skills, are mispositioned, increasing the text field width should fix it
    Image

  • the card in the location management page and the page heading is misaligned, this similar issues of padding and heading getting misalinged is observed in other pages as well such as bed management
    Image

  • Only show one loader for the entire page, rather than individual loader for each card in the facility location page
    Image

  • Remove the green bg for the collapse chevron in the sidebar. Just keep the chevron as black, and add a light gray bg on hover.

Image

  • add border colour to the border of covid details section, since the separation is not properly visible

Image

@AdityaJ2305
Copy link
Contributor

Can You assign me this issue please ?

@shivankacker
Copy link
Member Author

@AdityaJ2305 assigned

@bodhish
Copy link
Member

bodhish commented Oct 8, 2024

@AdityaJ2305 I will take this up

@AdityaJ2305
Copy link
Contributor

@bodhish sure

@nihal467 nihal467 changed the title Signout button icon is misaligned Bugs Detected After Merging PR #8737 , PR #8706, PR #8680 Oct 8, 2024
@gigincg
Copy link
Member

gigincg commented Oct 11, 2024

@nihal467 Can you double check if all of these items are a direct result of the mentioned PRs. And are not issues in Staging/Production.

@nihal467
Copy link
Member

nihal467 commented Oct 11, 2024

@gigincg moved all bugs out of this issue which are not caused by merging the mentioned three PR's

@nihal467 nihal467 removed their assignment Oct 11, 2024
@rithviknishad rithviknishad linked a pull request Oct 14, 2024 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Review required
Development

Successfully merging a pull request may close this issue.

6 participants