Skip to content

docs(DockManager): Add missing UnpinnedChanged event #3184

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

Merged
merged 2 commits into from
Aug 14, 2025

Conversation

dimodi
Copy link
Contributor

@dimodi dimodi commented Aug 14, 2025

No description provided.

@dimodi dimodi added the merge-to-production Use this label to get a comment to choose whether to merge the PR to production label Aug 14, 2025
@dimodi dimodi requested review from a team as code owners August 14, 2025 07:24
Copy link
Contributor

Hello @dimodi,

Check the below option if you would like to automatically generate PR to production. The automation uses the branch for the cherry-pick, and then will delete the branch. Please, do not delete it manually.

  • create PR to production

@dimodi dimodi merged commit a8f0f6c into master Aug 14, 2025
Copy link
Contributor

Here you are the link to your PR to production: #3185

@github-actions github-actions bot deleted the dock-unpinnedchanged branch August 14, 2025 07:28
Copy link
Contributor

@ikoevska ikoevska left a comment

Choose a reason for hiding this comment

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

Reviewed. Some typos are present.


## SizeChanged

The `SizeChanged` event is triggered when the `Size` parameter of the corresponding pane is changed.
The `SizeChanged` event fireswhen the `Size` parameter of the corresponding pane changes.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The `SizeChanged` event fireswhen the `Size` parameter of the corresponding pane changes.
The `SizeChanged` event fires when the `Size` parameter of the corresponding pane changes.


## UnpinnedChanged

The `UnpinnedChanged` event fireswhen the `Unpinned` parameter of the corresponding pane changes.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The `UnpinnedChanged` event fireswhen the `Unpinned` parameter of the corresponding pane changes.
The `UnpinnedChanged` event fires when the `Unpinned` parameter of the corresponding pane changes.


## OnPaneResize

The `OnPaneResize` event is fired when a pane is resized, except unpinned panes. It lets you respond to that change if needed - for example, call the `.Refresh()` method of a chart or otherwise repaint a child component in the content. You can also use it to, for example, update the saved [state](slug:dockmanager-state) for your users.
The `OnPaneResize` event fires when a pane is resized, except unpinned panes. It lets you respond to that change if needed - for example, call the `.Refresh()` method of a chart or otherwise repaint a child component in the content. You can also use it to, for example, update the saved [state](slug:dockmanager-state) for your users.
Copy link
Contributor

Choose a reason for hiding this comment

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

The first sentence sounds very unclear. I recommend revising it for clarity and better readability.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-to-production Use this label to get a comment to choose whether to merge the PR to production
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants