Skip to content

refactor(grids): change button text in column pinning both side samples#973

Merged
dkamburov merged 2 commits into
vnextfrom
iminchev/pinning-both-sides-buttons
Dec 19, 2025
Merged

refactor(grids): change button text in column pinning both side samples#973
dkamburov merged 2 commits into
vnextfrom
iminchev/pinning-both-sides-buttons

Conversation

@IMinchev64
Copy link
Copy Markdown
Contributor

Closes #3784

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates button labels in column pinning samples to better clarify that the actions apply to selected columns rather than a single column. The changes improve user experience by making button functionality more explicit.

Key Changes:

  • Updated button text from "Unpin Column" to "Unpin Selected Columns"
  • Updated button text from "Pin Left/Right" to "Pin Selected Left/Right"
  • Standardized quote usage in getElementById calls

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
samples/grids/tree-grid/column-pinning-both-sides/src/index.tsx Updated button labels to indicate they operate on selected columns and standardized quotes
samples/grids/hierarchical-grid/column-pinning-both-sides/src/index.tsx Updated button labels to indicate they operate on selected columns and standardized quotes
samples/grids/grid/column-pinning-both-sides/src/index.tsx Updated button labels to indicate they operate on selected columns and standardized quotes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MarielaTihova MarielaTihova added status: verified The PR is tested and ready for a merge and removed awaiting-test labels Dec 18, 2025
@dkamburov dkamburov merged commit fd2e9d2 into vnext Dec 19, 2025
5 checks passed
@dkamburov dkamburov deleted the iminchev/pinning-both-sides-buttons branch December 19, 2025 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: verified The PR is tested and ready for a merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve UX in column pinning both sides samples

4 participants