Skip to content

Ttonev/fix both side pin sample#3761

Merged
dkamburov merged 13 commits intovnextfrom
ttonev/fix-both-side-pin-sample
Jan 22, 2026
Merged

Ttonev/fix both side pin sample#3761
dkamburov merged 13 commits intovnextfrom
ttonev/fix-both-side-pin-sample

Conversation

@tishko0
Copy link
Contributor

@tishko0 tishko0 commented Oct 27, 2025

fixes #3758

@tishko0 tishko0 changed the base branch from master to vnext October 27, 2025 12:15
ChronosSF and others added 6 commits November 26, 2025 18:08
Updating Master with changes from Vnext
Updating prod with changes from staging
Updating latest fixes from staging to prod
…ail-chart-3823-master

fix(lob): import IgxCategoryChartModule instead of IgxCategoryChartCoreModule - master
@MarielaTihova
Copy link
Contributor

Looks good for me now. But please update the branch and also maybe try the approach that we discussed with you and @MayaKirova


public pinLeft() {
this.grid1.selectedColumns().forEach((col: IgxColumnComponent) => {
col.pinned = false;
Copy link
Contributor

Choose a reason for hiding this comment

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

Could just use the api instead of changing the props. The column has pin method API.

Copy link
Contributor

Choose a reason for hiding this comment

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


public pinLeft() {
this.grid1.selectedColumns().forEach((col: IgxColumnComponent) => {
col.pinned = false;
Copy link
Contributor

Choose a reason for hiding this comment

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

Also here pls


public pinLeft() {
this.grid1.selectedColumns().forEach((col: IgxColumnComponent) => {
col.pinned = false;
Copy link
Contributor

Choose a reason for hiding this comment

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

Also here pls

@dkamburov dkamburov merged commit 00050ed into vnext Jan 22, 2026
4 checks passed
@dkamburov dkamburov deleted the ttonev/fix-both-side-pin-sample branch January 22, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pinning on Both sides sample issues and improvements

6 participants