Patient Banner

[JNygaard-Skylight]

PULL REQUEST

Summary

A sticky banner displaying the patient's name and date of birth is now shown. If the environmental variable STANDALONE_VIEWER is truthy, the banner will not be shown. Otherwise, it will be visible.

Related Issue

Fixes #2185

Additional Information

Anything else the review team should know?

Checklist

• If this code affects the other scrum team, have they been notified? (In Slack, as reviewers, etc.)

[JNygaard-Skylight]

@sarahtress this is ready for design review:

[sarahtress]

@JNygaard-Skylight would it be out of scope for this ticket to change the header to align with the patient banner? like this:

[JNygaard-Skylight]

@JNygaard-Skylight would it be out of scope for this ticket to change the header to align with the patient banner? like this:

I can do that with this PR.

[JNygaard-Skylight]

@sarahtress Does this look better?

[sarahtress]

@JNygaard-Skylight yes! except, the font sizes look off between the patient banner and the header. patient banner should be 1.375rem while the font size for the header should be 2 rem. could you also adjust that? and then should be all good to go!

[JNygaard-Skylight]

@sarahtress how is this?

[sarahtress]

@JNygaard-Skylight looks great! thank you!

[BobanL]

Left a couple of comments. Could we also use rem instead of px on whatever is new?

[JNygaard-Skylight]

@sarahtress I made @BobanL's fixes, does it still look good:

[sarahtress]

looks good to me!

[BobanL]

🎆