Click on Appointment Time to Edit

yehster wrote on Monday, November 12, 2012:

https://github.com/yehster/openemr/commit/a37952c4dc4ed63ac633b87654f575fb7823c809
New feature that turns appointment times displayed on the calendar into clickable anchors which bring up the event for editing.

The existing double click functionality is unchanged, but I think this is a more natural way.  Also double click events don’t fire properly on tablets, so this provides an alternate mechanism.

-Kevin Yeh
kevin.y@integralemr.com

sunsetsystems wrote on Monday, November 12, 2012:

Not sure I understand… what calendar style is that?  I thought clicking on the event time already does that?

Rod
www.sunsetsystems.com

yehster wrote on Monday, November 12, 2012:

this affects the ajax_template(outlook style) calendar.

Try it out on:
http://demo.open-emr.org:2102/openemr/interface/login/login_frame.php?site=default

Single clicking an event does nothing currently. 

yehster wrote on Monday, November 12, 2012:

https://sourceforge.net/projects/openemr/forums/forum/202505/topic/5883690
And here is a report of “unexpected behavior” related to this issue.
FSGL said:

New bug: can’t single click on patient’s name to edit, have to go to Demographic page and click on the appointment to fix appointment. Not a big deal but would welcome a future patch.

Brady said:

For the appt edit, can you double click on the time of appt in calendar to edit it?

fsgl responds:

In the previous version, a single click on the appointment time brought up the dialog box for editing. In 4.1.1 a double click is necessary for editing.

bradymiller wrote on Monday, November 12, 2012:

Hi,

Sounds good to re-introduce the previous behavior (single click to edit). Suggest testing it on several browsers since I’ve noted the calendar stuff can sometimes be a bit finicky on different browsers.

-brady
OpenEMR

yehster wrote on Tuesday, November 13, 2012:

https://github.com/yehster/openemr/commit/6189f56dce9d545a30fbcb3a34ce12de93c8a9da
Added comments about choice not to escape htmlentities in create_event_time_anchor
pushed to master.
Tested on IE9, Firefox, Chrome and iOS5 safari.
-Kevin Yeh
kevin.y@integralemr.com