Skip to content

refactor: Column moving behavior #10506

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 32 commits into from
Jan 4, 2022
Merged

Conversation

rkaraivanov
Copy link
Member

@rkaraivanov rkaraivanov commented Nov 15, 2021

Closes #10176

Additional information (check all that apply):

  • Bug fix
  • New functionality
  • Documentation
  • Demos
  • CI/CD

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code (test guidelines)
  • This PR includes API docs for newly added methods/properties (api docs guidelines)
  • This PR includes feature/README.MD updates for the feature docs
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes ng update migrations for the breaking changes (migrations guidelines)
  • This PR includes behavioral changes and the feature specification has been updated with them

@teodosiah teodosiah added the 🛠️ status: in-development Issues and PRs with active development on them label Nov 15, 2021
@mmart1n mmart1n force-pushed the rkaraivanov/column-moving-grid-prop branch from b2233a1 to 730797e Compare November 17, 2021 12:37
@zdrawku zdrawku changed the base branch from master to 13.0.x November 25, 2021 12:18
@teodosiah teodosiah added ❌ status: awaiting-test PRs awaiting manual verification and removed 🛠️ status: in-development Issues and PRs with active development on them labels Dec 9, 2021
@teodosiah teodosiah requested a review from kdinev December 13, 2021 12:24
kdinev
kdinev previously approved these changes Dec 16, 2021
@ddaribo ddaribo added 💥 status: in-test PRs currently being tested ✅ status: verified Applies to PRs that have passed manual verification and removed ❌ status: awaiting-test PRs awaiting manual verification 💥 status: in-test PRs currently being tested labels Dec 17, 2021
@rkaraivanov rkaraivanov changed the base branch from 13.0.x to master December 17, 2021 11:31
@rkaraivanov rkaraivanov dismissed kdinev’s stale review December 17, 2021 11:31

The base branch was changed.

@teodosiah teodosiah requested a review from kdinev December 23, 2021 09:48
@@ -424,6 +424,9 @@ export class IgxColumnComponent implements AfterContentInit, OnDestroy, ColumnTy
/**
* Sets/gets whether the column is movable.
* Default value is `false`.
*
* @deprecated in version 13.1.0. Use `IgxGridComponent.moving` instead.
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't this be at the beginning of the comment?

kdinev
kdinev previously approved these changes Jan 4, 2022
@kdinev kdinev merged commit 075bfee into master Jan 4, 2022
@kdinev kdinev deleted the rkaraivanov/column-moving-grid-prop branch January 4, 2022 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
grid: column-moving version: 13.1.x ✅ status: verified Applies to PRs that have passed manual verification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Why column moving feature is enabled on a per-column level?
6 participants