Skip to content

Modal : Added Titlebar , title , X handling , resizable and movable for windows #14636

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

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

iamAbhi-916
Copy link
Contributor

@iamAbhi-916 iamAbhi-916 commented May 2, 2025

Description

Added Titlebar , title , X handling , resizable and movable for windows

Type of Change

  • New feature (non-breaking change which adds functionality)

Why

What is the motivation for this change? Add a few sentences describing the context and overall goals of the pull request's commits.

Resolves [Add Relevant Issue Here]
#14067

What

Added JS side hadndling for windows only prop Title .
Passed the prop from js side to native side and handled the behaviour if title has been passed and not
handled closing modal using X
modal is resizable
modal is movable
the title can be updated dynamically.

Screenshots

modal_new.mp4
modal_moves.mp4

Testing

tested in playground

Changelog

yes

Add a brief summary of the change to use in the release notes for the next release.
Added support for a title prop on Windows Modals, allowing a title bar to be displayed.
The modal window can now be closed using the 'X' button, triggering the onRequestClose event.
Modals with a title bar are now resizable, and the title can be updated dynamically.

Microsoft Reviewers: Open in CodeFlow

@iamAbhi-916 iamAbhi-916 self-assigned this May 2, 2025
@iamAbhi-916 iamAbhi-916 marked this pull request as ready for review May 3, 2025 11:29
@iamAbhi-916 iamAbhi-916 requested a review from a team as a code owner May 3, 2025 11:29
@iamAbhi-916 iamAbhi-916 marked this pull request as draft May 3, 2025 11:36
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