Open menu should support background scrolling
- 主要言語
- TypeScript
- スター
- 25k
- フォーク
- 6.8k
- 平均マージ
- 1日 8時間
- マージ済み PR(30日)
- 91
説明
#### Bug, feature request, or proposal:
Bug? Reading the source, when having a menu open, I believe the background should be able to be scrolled using the mouse. The background can be scrolled with the arrow keys.
#### What is the expected behavior?
The background can scroll.
#### What is the current behavior?
The background cannot scroll.
#### What are the steps to reproduce?
1. Visit https://material.angular.io/components/menu/overview
2. Open the menu by clicking the trigger: "Menu"
3. Attempt to scroll the page (not the menu) using the mouse.
#### What is the use-case or motivation for changing an existing behavior?
A dialog box should block background scrolling, but the menu should not.
#### Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Current
#### Is there anything else we should know?
It is blocked by `pointer-events: auto`: https://github.com/angular/material2/blob/4d82f833c7e4a22814b8e225f05ef4ba01a86ad6/src/cdk/overlay/_overlay.scss#L61.
The menu does have the scroll strategy of reposition, which I believe should mean that the background should be able to be scrolled: https://github.com/angular/material2/blob/fe37cb2551929844d5467e1b9bf45032df35981f/src/lib/menu/menu-trigger.ts#L57
コントリビューションガイド
調査の方向性
Material メニューの概要 URL で問題を再現し、その後 src/cdk/overlay/_overlay.scss の pointer-events 付近と、src/lib/menu/menu-trigger.ts のメニューのスクロール戦略付近を読んでください。メニューが開いている間にマウスでスクロールするとページが移動することを、レポートに記載されたダイアログの動作を変更せずに確認してください。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- angular, typescript
- 領域
- frontend
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 静か
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 55/100