Reference application 2.10-SNAPHOT UI testing

I get you @dkayiwa on this! Can we attach some screenshots where possible, especially if there is a discrepancy?

@irenyak1

Yes that will be great if you can attach and let know how to reproduce the issue so can attend the issues soon :slightly_smiling_face:

The very first one I have found is on the home page. The word ’ Find Patient Record’ occupies two lines in this RA UI as opposed to one line in the main UI. This is for RA 2.10SNAPSHOT. The icons look a slightly smaller than those in the usual UI, May be this explains why the words “Find Patient Record” flow to the second line.

Below is the usual wording for the Find Patient Record icon

Second is there RA 2.10-SNAPSHOT

In the one above, the search bar looks incomplete without the cancel icon in it and also there’s a big space between the search bar and the table where the search results are displayed. You may want to reduce that space.

As compared to our usual UI

@irenyak1 this is awesome! :slight_smile:

@mozzy do you now see why it is important to fully test the entire reference application?

3 Likes

RA 2.10 SNAPSHOT. This section seems to be lacking the Search field and the filters on the extreme right seem to be so close to the result table, I think they are occupying the place for the search field. You may want to edit this. Also even if this shows no results for the active visits, if you are keen you will realize that the word none spans through 4 coloumns. Please edit so that each table header has a clear column for it’s table data like it is in the Usual UI image .

Our usual UI

Hi @irenyak1

The first 2 issues you mentioned now fixed.

Please look in to the video

1 Like

This is actually not a bug.When you dont have active visits we dont have anything to search so the search box automatically hides.Starta visit with a patient and it will work.

I already added an active visit so it should be fine by now on the qa-refapp-server :slightly_smiling_face:

1 Like

cc :- @dkayiwa @mozzy

RA 2.10 SNAPSHOT

Under Data management, The left margin for ‘Merge Patient Electronic Records’ icon is too big

compared to the Usual UI.

You may want to reduce the left margin to match the usual one.

Issue 1 :-

PR :- https://github.com/openmrs/openmrs-module-coreapps/pull/248

Issue 2 :-

PR :- https://github.com/openmrs/openmrs-module-referenceapplication/pull/70

Issue 3

PR :- https://github.com/openmrs/openmrs-module-coreapps/pull/248

cc :- @mksd , @mozzy can we test and merge the fixes :slightly_smiling_face:

RA 2.10 SNAPSHOT Merge Patient Electronic Records, The bar codes in the Patient ID fields are a bit not well aligned with the inputs and also this page should not be scrollable. There’s a scroll bar on the right as you can see. You may want to remove it.

as compared to the usual UI

Hi @irenyak1

I cant reproduce the scrollable issue on the qa-ref app in main 2 browsers Firefox and Google chrome.

Issue 4 :-

Fix :- PR’s

@irenyak1 am just so impressed by how much sensitive to details you can be , good work. And @ayesh am also amazed at how fast you fix the issues …good job :grinning: .

2 Likes

Hello @mozzy I have tried resetting this demo server but it is still down. Could you help me out. I just need to do some more UI testing. Mozzy have you been able to merge the PRs from @ayesh?

@irenyak1 you can request access to bamboo if you do not already have it. Then you can restart the demo server by simply running this CI plan: https://ci.openmrs.org/browse/OENV-RD/latest

@dkayiwa the server is still down even after restarting .I actually have access to bamboo.