Ismailtaktak / freemedforms

Automatically exported from code.google.com/p/freemedforms
Other
0 stars 0 forks source link

PMHx: pmhx creator dialog needs update #205

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
This dialog is too big. We need to update the UI (remove the name of the 
patient...).

Original issue reported on code.google.com by eric.mae...@gmail.com on 7 Nov 2012 at 1:19

GoogleCodeExporter commented 9 years ago
i can do this if you want.  i like UI tasks. 

Original comment by christian.a.reiter@gmail.com on 7 Nov 2012 at 5:36

GoogleCodeExporter commented 9 years ago
Sure bug as no status. When I start working on something, I change the status 
of the issue...
That's yours :D

Original comment by eric.mae...@gmail.com on 7 Nov 2012 at 5:39

GoogleCodeExporter commented 9 years ago
Updated, please check.

Original comment by christian.a.reiter@gmail.com on 8 Nov 2012 at 9:19

GoogleCodeExporter commented 9 years ago
ATM I have implemented a check, if there is a patient() - if, it displays the 
patient name as title, if not, it just displays the PMHx general title.
BUT: is there ANY case where you can open the PMHx creator dialog witn NO 
patient opened??
I think not!?

So I would suggest to Q_ASSERT patient() in the PMHViewer ctor as it needs a 
patient anyhow.

ok?

Original comment by christian.a.reiter@gmail.com on 9 Nov 2012 at 10:32

GoogleCodeExporter commented 9 years ago
Good start. Thanks

Why not:
- remove the labels for "label", "date"
- put labelLineEdit && date next to next into one unique groupbox
- create a grouppbox for type/status. Add in the combos "Type - XXX" "Status - 
XXX". Remove their labels

Original comment by eric.mae...@gmail.com on 10 Nov 2012 at 9:57

GoogleCodeExporter commented 9 years ago
oh, I forgot to push my second changes, please have a look.

Original comment by christian.a.reiter@gmail.com on 10 Nov 2012 at 12:06

GoogleCodeExporter commented 9 years ago

Original comment by eric.mae...@gmail.com on 15 Sep 2013 at 5:09