update patient form (general information) UX

[vault-developer]

🚀 Feature Proposal

Update general information form UX

1. Put "Unknown" checkbox under "Date of Birth" field.

Motivation

Now we have input "Date of birth" and checkbox "Unknown"

1. It's not clear, that checkbox "unknown" is related to field "Date of birth". User can suggest, that this checkbox is for "Patient type", because it's under this field.
2. If "Unknown" is checked, "Date of Birth" field is hidden. If someone will view such patient, there is no way to find out, what field does "Unknown" belong to.

P.S. I can work on this issue if you don't mind

[matteovivona]

The date of birth isn't a mandatory field, so if you don't enter it, it is in fact already "Unknown".
@lauggh @jackcmeyer @fox1t opinions? does this new checkbox come in useful?

[lauggh]

Yes. I agree with the proposal here. While not required, it is useful as a checklist of questions during data entry or registration

[fox1t]

I agree too with the proposal. Actually this makes DOB mandatory. So let’s add also it to this proposal.

[blestab]

Do not hide "Date of Birth" field if "Unknown" is checked, only disable it.

I am not sure about this proposal as checking "Unknown" shows the Approximate Age (at least in staging), which takes an integer and i thought the idea here was to calculate an approximate Date of Birth based on the Approximate Age?

On Editing the patient, the Date of birth field now has a value

Perhaps what we need is helper text to tell the user of the relationship between Date Of Birth, Unknown and Approximate Age fields?

[jackcmeyer]

To close the loop on this:

I think the final requirement is merely a UI change, rather than a functionality change. The functionality we have now where checking unknown changes the input box to an integer input box for an estimate age is the desired behavior.