angular / angular/components

Open menu should support background scrolling

オープン
#6,927 コメント 9 件 リアクション 12 件 担当者 0 名 GitHub で見る
area: material/menu P4
主要言語
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。