Cold Chain: improve the appearance of temperature breach notification - Githubissues

msupply-foundation / open-msupply

Open mSupply represents our most recent advancement in the Logistics Management Information System (LMIS), expanding on more than two decades of development inherited from the well-established legacy of the original mSupply.

https://msupply.foundation/open-msupply/

Other

20 stars 12 forks source link

Cold Chain: improve the appearance of temperature breach notification #3293

Open marthakeezy opened 5 months ago

marthakeezy commented 5 months ago

Is your feature request related to a problem? Please describe 👀

When logging into omSupply, at the top of the screen we can see a temperature breach notification. The notification looks misaligned - the text appears too close to the top of the screen, and the button 'View Details' overlaps with a bottom line of the notification frame.
'Total unacknowledged: 6' part of the notification looks misaligned
Last temperature reading includes a space between the number and the degree symbol

Describe the solution you'd like 🎁

If possible, maybe the text of the notification can be moved down (to leave space between the top of the screen and the text), and the button can be moved up (to leave space between the button and the bottom of the notification).

screenshot-1710451620507

'Total unacknowledged: 6' part of the notification looks misaligned - would it be better to move it, so it starts either where the breach icon starts or where the text 'Temperature breach detected' starts? So it's not sitting in the middle.

screenshot-1710451657324

To make it consistent with #3055, we should remove the space between digits and degree symbols

Describe alternatives you've considered 💭

N/A

Additional context 💌

N/A

Moneyworks Jobcode 🧰

N/A

roxy-dao commented 5 months ago

A bit annoying to look at but doesn't block workflow