Skip to content

Conversation

@artnim
Copy link
Contributor

@artnim artnim commented Oct 29, 2025

This PR introduce a resizing option to the Side Dialog (fixes #2346):

demo

@artnim artnim marked this pull request as draft October 30, 2025 11:02
@artnim artnim force-pushed the side-dialog-resizable branch from 09ba7e9 to 2fcdbd7 Compare October 30, 2025 15:52
@artnim artnim marked this pull request as ready for review October 30, 2025 15:53
@artnim artnim marked this pull request as draft November 5, 2025 09:57
@artnim artnim force-pushed the side-dialog-resizable branch from b12c9ad to 4d4aa08 Compare November 5, 2025 15:44
@artnim artnim force-pushed the side-dialog-resizable branch from 4d4aa08 to 3b20fc0 Compare November 5, 2025 20:01
@artnim artnim marked this pull request as ready for review November 5, 2025 20:01
@MPapst
Copy link
Contributor

MPapst commented Nov 26, 2025

Hi @enchev / @akorchev,

does that fit into your roadmap? This is a really good extension to the side dialog.

Copy link
Collaborator

@akorchev akorchev left a comment

Choose a reason for hiding this comment

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

Hi @artnim,

Thank you for the pull request. I have a few requests / questions which you can find here.

Move code back to RadzenDialog.razor.
- Take position from options.
- Take width and height from sideDialogs clientWidth and clientHeight.
- Take minWidth and minHeight from options.
- Remove superflous pointer capturing.
- Remove superflous 'dragging' class.
@artnim artnim requested a review from akorchev November 26, 2025 20:35
@artnim artnim requested a review from akorchev November 27, 2025 15:49
@akorchev
Copy link
Collaborator

Thank you for your commitment @artnim!

The latest changes are top-notch. I will ask @yordanov to review the SASS style of things and we will merge the PR.

@akorchev akorchev merged commit f422573 into radzenhq:master Dec 3, 2025
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.

Feature: Optional, re-sizable side dialog.

4 participants