openhealthcare / elcid

elCID deployment for UCLH
http://elcid.openhealthcare.org.uk
Other
7 stars 3 forks source link

Walkin review list displays full clinical advice entry #1536

Closed GabPoll closed 6 years ago

GabPoll commented 6 years ago

Should only show minimal info, like OPAT current list, for example - i.e. date, name of person only. Then can click on entry to read content

GabPoll commented 6 years ago

Same for walkin doctor list. Takes too much space by displaying all the advice.

GabPoll commented 6 years ago

would be assisted if it went in hand in hand with #1535, as then you could scan a patient but then complete their letter and discharge them within in patient detail view

pacharanero commented 6 years ago

WIP on this ticket at https://github.com/openhealthcare/elcid/pull/1563

davidmiller commented 6 years ago

@GabPoll see the Heroku server https://elcid-marcus-test.herokuapp.com/ for the current implementation...

I have mixed feelings about the ellipsis - it's an entirely new patten for us... Maybe try adding some realistic Clinical advice entries and seeing how they feel ?

GabPoll commented 6 years ago

personally i like the new implementation but there is an issue that i may have mentioned before but forgot to github. The nursing staff write a lot also in Symptoms Details box, which can have same effect. Could we implement in same way as clinical advice please? image

GabPoll commented 6 years ago

i checked and it seems fine, but i can be sure there is no truncation on patient details view? Seems to be only on list view to me

GabPoll commented 6 years ago

the ellipsis is a bit subtle, i have to be honest @davidmiller . It may not be obvious there is more to read A more obvious "MORE" in bold, which is clickable?

pacharanero commented 6 years ago

We are currently blocked on deployment to Test because of problems with the VPN token, but the PR for this work has been merged https://github.com/openhealthcare/elcid/pull/1563