Skip to content

Conversation

@Harshraj112
Copy link

Requirements

This PR has a title that briefly describes the work done including the ticket number and follows the conventional commit format.

My work is based on existing Patient Chart and Vitals & Biometrics UI behavior and follows the O3 style guide.

My work was manually validated using existing application behavior across different screen sizes.

Summary

This PR fixes a UI issue where the primary “Add” (+) action in the Vitals & Biometrics section becomes partially hidden when the “Start a visit” right-side drawer is opened on the Patient Chart.

When the drawer opens, the main content area shrinks but does not correctly reflow its contents, causing the primary action button to be clipped.
This change ensures the main content resizes properly and keeps the primary action fully visible and accessible even when the right-side drawer is open.

Screenshots

Before:
Screenshot 2025-12-21 at 2 37 38 AM

After:
Screenshot 2025-12-24 at 12 53 42 AM
Screenshot 2025-12-24 at 12 53 54 AM

Related Issue

O3-5297

Other

Improves usability and discoverability of a primary clinical action

Prevents user confusion, especially on smaller screens and laptops

Aligns Patient Chart layout behavior with responsive UI best practices

/* Main content + right workspace panel.
When workspace is closed: second column collapses to 0, first column uses 100% width.
When workspace is open: first column shrinks, second column expands up to min(40rem, 50vw).
Use auto for second column to allow it to collapse to 0 when empty. */
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Harshraj112, the SCSS comments are too much detailed and breaks consistency with the rest of the codebase

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay! So this cause any prbm? I have make it like rest of the codebase?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this doesn't cause any problem , but to maintain code consistency u can remove the comments

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants