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

[igxFor] - Scroll not working properly #10949

Closed
lalo-mx opened this issue Feb 3, 2022 · 1 comment · Fixed by #11009
Closed

[igxFor] - Scroll not working properly #10949

lalo-mx opened this issue Feb 3, 2022 · 1 comment · Fixed by #11009
Assignees
@ChronosSF
Labels
🐛 bug Any issue that describes a bug scroll version: 13.0.x ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.

Comments

@lalo-mx
Copy link

lalo-mx commented Feb 3, 2022

Description

Scroll not working properly

There is bug with a list using igxFor with dynamic items(displayed only on mouse over) that does not allow to scroll down in the list.

  • igniteui-angular version: 13.0.10
  • browser: 97.0.4692.99

Steps to reproduce

  1. Open example https://stackblitz.com/edit/angular-smnpt7
  2. Try to scroll with mouse over the names

Related:

Result

The scroll does not work.

Expected result

The scroll should works.

Attachments

It works with 13.0.0
angular-smnpt7.zip
@lalo-mx lalo-mx added the 🐛 bug Any issue that describes a bug label Feb 3, 2022
@lalo-mx
Copy link
Author

lalo-mx commented Feb 8, 2022 •
edited
Loading

Workaround - It could break easily with any new release:

@ChronosSF ChronosSF assigned ChronosSF and unassigned dkamburov Feb 9, 2022
@ChronosSF ChronosSF added 🛠️ status: in-development Issues and PRs with active development on them scroll version: 13.0.x and removed 🆕 status: new labels Feb 9, 2022
ChronosSF added a commit that referenced this issue Feb 15, 2022
@ChronosSF
fix(scroll): adding overflow check for inner scorlls #10949
b26ef34
ChronosSF added a commit that referenced this issue Feb 15, 2022
@ChronosSF
fix(scroll): adding overflow check for inner scorlls #10949
d8faa29
@ChronosSF ChronosSF mentioned this issue Feb 15, 2022
Merged
14 tasks
@ChronosSF ChronosSF 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 Feb 15, 2022
@ChronosSF ChronosSF mentioned this issue Feb 15, 2022
Merged
14 tasks
dkamburov added a commit that referenced this issue Feb 17, 2022
@dkamburov
Merge pull request #11009 from IgniteUI/sstoychev/inner-scroll-extra-…
33b44d5 
…master

fix(scroll): adding overflow check for inner scorlls #10949 - master
dkamburov added a commit that referenced this issue Feb 17, 2022
@dkamburov
Merge pull request #11010 from IgniteUI/sstoychev/inner-scroll-extra-…
8f5be4e 
…13.0

fix(scroll): adding overflow check for inner scorlls #10949 - 13.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ChronosSF ChronosSF

Labels
🐛 bug Any issue that describes a bug scroll version: 13.0.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.

fix(scroll): adding overflow check for inner scorlls #10949 - master IgniteUI/igniteui-angular
3 participants
@dkamburov @ChronosSF @lalo-mx