Patient summary gui mods code review

bradymiller wrote on Wednesday, August 04, 2010:

hey,

Made several modifications to the patient summary gui.

1) Created mechanism (using sessions variables, ajax, and specific user entries in the users table) to make the expand/collapse selections in the patient summary persistent (even when logout/login) for each user.
2) Clarified buttons (edit buttons for each entry)
3) Migrated prescription link (removed rx from top and put with the Presciption entry on right)
4) Removed modul from the document link (the modul for this specific screen is currently not useful)

patch here:
http://sourceforge.net/tracker/?func=detail&aid=3039226&group_id=60081&atid=1245239#

github branch here:
http://github.com/bradymiller/openemr/commits/gui-improve

Please review/test if desired and feel free to flame away (especially if there are better ways to make the expand/collapse entries persistent).

thanks,
brady

bradymiller wrote on Wednesday, August 04, 2010:

hey,

Posted a new revision (patient_summary_2.patch in above tracker link), since I relaized do not need to use session
variables. This tests well and is what I plan to commit this evening or tomorrow.

The github branch is here:
http://github.com/bradymiller/openemr/commits/gui-improve_2

-brady

bradymiller wrote on Thursday, August 05, 2010:

hey,
Committed this patch and restarted the cvs demo. Let me know your thoughts. Goal is to get the gui ready for 4.0 release as quick as possible.

In case you want to see the code, here’s the link to the commit:
http://github.com/openemr/openemr/commit/dcff0e1f6cc819f543b5486e9b52850275992cf4

-brady

bradymiller wrote on Saturday, August 07, 2010:

hey,

Committed some more work on this topic to finish up above:
http://github.com/openemr/openemr/commit/8f5f089cb71c505af60de3c1136c56699329f1a8

It’s currently in the cvs demo if you want to try it out.

What I did:
1) Made sections and links uniform (made a widget for this)
2) Refined the mechanism that keeps the expand/collapse
   selections for each user persistent. Created a user_settings
   table to hold this information, which can also be used
   in future for other user specific settings.
3) Allergies are now shown in bright red

-brady

johnbwilliams wrote on Saturday, August 07, 2010:

Brady,

We will look at this and comment.  The CCR Summary and CCD(L3) Summary options (currently under development by a MU team) must be in the 4.0 release as well.

John