lh-ehr icon indicating copy to clipboard operation
lh-ehr copied to clipboard

Add appointment's tooltip display styles

Open apooravc opened this issue 6 years ago • 10 comments

Added 4 display styles for an appointment's tooltip, like an appointment's slot text has. Also, in case of provider events, there wasn't a check for including comment, if there is one, in event's tooltip which is added in this commit.

@teryhill Please have a look. I also noticed that the event's category+comment becomes a part of hyperlink along with name, probably because of including them in $e['title'] instead of $e['e_info'] in switch cases in get_provider_event.php: text

Could you test if it's appearing like above on your machine as well? Including status, name in hyperlink or in $e['title'] and rest - event's category+comment in $e['e_info'] would separate them and all of them would be governed as per Appointment Display Style global.

apooravc avatar Jul 01 '18 18:07 apooravc

I will pull this today.

aethelwulffe avatar Jul 05 '18 14:07 aethelwulffe

We are looking to see what can be come up with for implementing tooltips in cool and useful ways.

aethelwulffe avatar Jul 13 '18 20:07 aethelwulffe

@teryhill @aethelwulffe There's a tooltip plugin qTip2 which has a cool layout as shown below: qtip

So, the details of an appointment could be represented in above layout with image of patient on left side and description of appointment with its details - category, status, comment, facility on right side. Should we use it or implement a layout like that internally?

apooravc avatar Jul 15 '18 20:07 apooravc

Can you bring the asset in first and let us see how large it is? Any other places that you can think to use it?

teryhill avatar Jul 17 '18 02:07 teryhill

@teryhill qTip's css file is 2 KB and js file is 26 KB and are added with last commit. It can be used as a default tooltip plugin across EHR for displaying simple info. texts (as in Globals), HTML with images, links (in Calendar) and it can also be used to fetch dynamic medical info./data from web via AJAX. Or we may improve the current tooltip layout used in Calendar.

Also, appt. display is changed from: text to: apptd so that only status and patient's name is in hyperlink, according to globals.

apooravc avatar Jul 17 '18 15:07 apooravc

Latest commit replaces native Calendar tooltip with qTip. Previews:

  1. Patient event tooltip with profile image and non-empty comment: pic1

  2. Patient event tooltip without profile image and empty comment: pic2

  3. Provider event tooltip with empty comment: pic3

In case of appointment (patient event) tooltips, content depends on Appointment Tooltip Display Style global which is set to show everything in this case. We could also add facility, billing facility in tooltip.

@teryhill Please have a look.

apooravc avatar Jul 27 '18 22:07 apooravc

We have, or have had qtip in the code before. This PR is pretty light. Testing now for basic operation.

With fullcalendar project, have you inquired with them as to using qtip in their project (and why they didn't)?

aethelwulffe avatar Jul 31 '18 17:07 aethelwulffe

I think the old qtip was in the ZEND stuff, so nothing to dump. In my minimal use-case test, this works nice. I have the outstanding question based on the fullcalendar project, but otherwise this seems real good to me.

aethelwulffe avatar Jul 31 '18 18:07 aethelwulffe

Would someone test this and do a review Pretty Please?

aethelwulffe avatar Jan 18 '19 19:01 aethelwulffe

Fixed conflicts here. Can I get a second review on this?

aethelwulffe avatar May 09 '19 14:05 aethelwulffe