Skip to content
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.

Sign up for GitHub

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

Jump to bottom

onColumnVisibilityChanged event is not fired when hiding a column through ESF. #4765

Closed
gedinakova opened this issue May 14, 2019 · 0 comments · Fixed by #4783
Closed

onColumnVisibilityChanged event is not fired when hiding a column through ESF. #4765

gedinakova opened this issue May 14, 2019 · 0 comments · Fixed by #4783
Assignees
@gedinakova
Labels
🐛 bug Any issue that describes a bug 🧨 severity: medium grid: excel-style-filtering version: 7.2.x version: 7.3.x ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.

Comments

@gedinakova
Copy link
Contributor

gedinakova commented May 14, 2019 •
edited
Loading

Description

onColumnVisibilityChanged event is not fired when hiding a column through ESF.

  • igniteui-angular version: 7.3.x, 7.2.
  • browser: N/A

Steps to reproduce

  1. Run https://stackblitz.com/edit/grid-columnvisibilitychanged
  2. Open Column Hiding UI and hide a column
  3. Check the console => ColumnVisibilityChanged is logged
  4. Open ESF and hide a column
  5. Check the console

Result

onColumnVisibilityChanged is not fired so 'ColumnVisibilityChanged' is not logged.

Expected result

onColumnVisibilityChanged event should be fired when hiding a column from any piece of UI.
@gedinakova gedinakova self-assigned this May 14, 2019
@gedinakova gedinakova added the 🛠️ status: in-development Issues and PRs with active development on them label May 14, 2019
gedinakova added a commit that referenced this issue May 15, 2019
@gedinakova
fix(ESF): #4765 Emit onColumnVisibilityChanged event on hide.
50d8bf5
gedinakova added a commit that referenced this issue May 15, 2019
@gedinakova
fix(ESF): #4765 Updated hiding test.
bd9f03d
@gedinakova gedinakova mentioned this issue May 15, 2019
Merged
14 tasks
gedinakova added a commit that referenced this issue May 15, 2019
@gedinakova
fix(ESF): #4765 Updated hiding test.
fc6c47e
gedinakova added a commit that referenced this issue May 15, 2019
@gedinakova
fix(ESF): #4765 Emit onColumnVisibilityChanged event on hide.
4e46707
@gedinakova gedinakova mentioned this issue May 15, 2019
Merged
14 tasks
@gedinakova gedinakova added ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged. and removed 🛠️ status: in-development Issues and PRs with active development on them labels May 15, 2019
gedinakova added a commit that referenced this issue May 17, 2019
@gedinakova
Emit onColumnVisibilityChanged when hiding a column through ESF UI. #…
711244b 
…4765 (#4791)

* fix(ESF): #4765 Emit onColumnVisibilityChanged event on hide.

* fix(ESF): #4765 Updated hiding test.
gedinakova added a commit that referenced this issue May 17, 2019
@gedinakova
Emit onColumnVisibilityChanged when hiding a column through ESF UI. #…
54f590e 
…4765 (#4792)

* fix(ESF): #4765 Updated hiding test.

* fix(ESF): #4765 Emit onColumnVisibilityChanged event on hide.
@kdinev kdinev mentioned this issue May 17, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gedinakova gedinakova

Labels
🐛 bug Any issue that describes a bug 🧨 severity: medium grid: excel-style-filtering version: 7.2.x version: 7.3.x ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Merging 7.3.x into master IgniteUI/igniteui-angular
1 participant
@gedinakova