Fix Patient documents

[mbayopanda]

This PR fix the patient document according the #475 issue.

• Fix the avatar image size to fill the box
• Use of actions buttons in document section for patient record details
• Fix the downloaded file extension
• Remove the preview buttons

Closes #475

[sfount]

@mbayopanda

I've experimented with your branch and this feature is looking great 👍.
I have a few comments about how the feature interacts with the overall page.

Component overflow
When there are two or more files attached to an individual patient the component overflows as it should. However the header also scrolls out of view which can lead to having no idea what the content in the middle of the page is.

Header overflowing with content

Component style
This component looks great but I think it would be best bring it in-line with the other components on the page - if you disagree then we can adjust the style of the other components. The most important thing is making sure everything looks and works in a reliably similar way.

Different component styles

My recommendations are:

• Content presented using a Panel panel panel-default as used by patient check in.
• Overflow only the content body to make sure the 'Patient Documents' title is always presented
• 'No patient documents' alert replaced with text-info, this is how information is presented to users throughout 2.x

Other than these small comments everything else works well for me. These are just recommendations so if there is a reason they are not practical that is not a problem. Otherwise the changes could be included in this pull request to close out the review issue.
@mbayopanda @jniles

[jniles]

This reads really nicely. The component API looks fantastic. Well done!

[jniles]

@mbayopanda, I've complete a review of the code. It's very good! If you are willing, #575 (comment) could be addressed before this is merged.

I agree with @sfount's analysis of the page. The bh-find-document and patient check in panel do not line up. Either in this PR or another, we should make the bh-find-document use a regular panel, instead of a custom CSS box.

[jniles]

Hmm.. Hardcoded styles are not a great idea. Is there anyway to make this component take up 100% height?

This won't block the PR, as it may require CSS changes beyond this component.

[jniles]

@mbayopanda, LGTM! Well done!