Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve Modal Layout: Group Shortcuts and Buttons with Responsive Scrolling. Solves #10591 #10612

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Deeptanshu-sankhwar
Copy link
Contributor

Solves #10591

  1. Changes in uiConfirm:
    Added a content-wrapper div to group the message-text and buttons sections under a single parent. This improves layout structure and enables scoped scrolling behavior.

  2. Changes in uiShortcuts:

Introduced a content-wrapper div to group the shortcuts list and buttons sections. This ensures consistent scrolling behavior and dynamic layout within the modal.

  1. CSS Updates:

Applied scrolling behavior (overflow-y: auto) to the content-wrapper for both modals.
Added max-height: calc(90vh - 100px) to dynamically allocate space for the header and footer, ensuring responsiveness across various screen sizes.

  1. Improved Structure and Responsiveness:

Scoped scrolling within the content-wrapper while keeping the header and footer static.
Ensured better organization and a clean layout to support future extensions.

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.

1 participant