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

popover blocks webpage scrolling in some layout #4763

Closed
rppig42 opened this issue Feb 9, 2020 · 1 comment · Fixed by #5925
Closed

popover blocks webpage scrolling in some layout #4763

rppig42 opened this issue Feb 9, 2020 · 1 comment · Fixed by #5925
Assignees
@wzhudev
Labels
🤚 Blocked By @angular/cdk Component: Popconfirm Component: Popover Component: Tooltip 💪 Enhancement
Milestone
blocked

Comments

@rppig42
Copy link
Contributor

rppig42 commented Feb 9, 2020

You can reproduce this issue by visiting this link https://stackblitz.com/edit/angular-hxlgre
@wzhudev
Copy link
Member

wzhudev commented Feb 9, 2020 •
edited
Loading

When we're using 'click' as the trigger, we would use a backdrop to detect "clicking outside" event. Unfortunately, this backdrop element would also make scroll events not dispatched to the body element. For now, you can use other triggers.

Since we're using Angular CDK, we will seek the opportunity to fix this on their side.

@weekly-digest weekly-digest bot mentioned this issue Feb 16, 2020
Closed
@hsuanxyz hsuanxyz added this to the blocked milestone Oct 10, 2020
@wenqi73 wenqi73 mentioned this issue Oct 15, 2020
Merged
3 tasks
wenqi73 added a commit to wenqi73/ng-zorro-antd that referenced this issue Oct 15, 2020
@wenqi73
refactor(module:all): remove the backdrop of most components
f481f32 
Related components:
autocomplete, cascader, datepicker, timepicker, mention, popconfirm, popover, tooltip, select, treeselect

close NG-ZORRO#4763
close NG-ZORRO#5061
close NG-ZORRO#4259
close NG-ZORRO#2508
wenqi73 added a commit to wenqi73/ng-zorro-antd that referenced this issue Oct 21, 2020
@wenqi73
refactor(module:all): remove the backdrop of most components
39a39e3 
Related components:
autocomplete, cascader, datepicker, timepicker, mention, popconfirm, popover, tooltip, select, treeselect

close NG-ZORRO#4763
close NG-ZORRO#5061
close NG-ZORRO#4259
close NG-ZORRO#2508
vthinkxie pushed a commit that referenced this issue Oct 21, 2020
@wenqi73
refactor(module:all): remove the backdrop of most components (#5925)
aa4ac2b 
* refactor(module:all): remove the backdrop of most components

Related components:
autocomplete, cascader, datepicker, timepicker, mention, popconfirm, popover, tooltip, select, treeselect

close #4763
close #5061
close #4259
close #2508

* chore: update codes

* chore: change cascader margin
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wzhudev wzhudev

Labels
🤚 Blocked By @angular/cdk Component: Popconfirm Component: Popover Component: Tooltip 💪 Enhancement
Projects
None yet
Milestone
blocked
Development

Successfully merging a pull request may close this issue.

refactor(module:all): remove the backdrop of most components wenqi73/ng-zorro-antd
3 participants
@wzhudev @hsuanxyz @rppig42