Skip to content
This repository has been archived by the owner on Oct 7, 2020. It is now read-only.

Enhanced mdc-select menu does not stay open on touch. #1762

Closed
Jordan-Amanda opened this issue Jan 22, 2019 · 2 comments · Fixed by #1787
Closed

Enhanced mdc-select menu does not stay open on touch. #1762

Jordan-Amanda opened this issue Jan 22, 2019 · 2 comments · Fixed by #1787
Labels
Milestone

Comments

@Jordan-Amanda
Copy link

Describe the bug
in the enhanced mdc-select menu does not handle touch taps. Like on android devices. it treats them like a double click so the mdc-menu only opens for fraction of a second then closes and does not remain open.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'https://trimox.github.io/angular-mdc-web/#/angular-mdc-web/select-demo/examples'
  2. Press 'F12' to open up chromes inspect menu.
  3. Click on 'Toggle device Toolbar' looks like:
    image
    this should make your cursor a touch dot instead of a mouse cursor.
  4. Scroll down to the 'Enhanced Select'
  5. Click on the word 'Fruit*' in the select
  6. See error

Expected behavior
Option Menu should stay open and let the user select other options in the menu, without closing right away.

Screenshots
If applicable, add screenshots to help explain your problem.

What Angular MDC version are you using? 0.43.4

What OS are you using?: Android and chrome.

What browser(s) is this bug affecting?: chrome and apps on android devices.

Additional context
Add any other context about the problem here.

@trimox trimox added the bug label Jan 23, 2019
@trimox
Copy link
Owner

trimox commented Jan 23, 2019

@Jordan-Amanda Thanks for reporting this!

Also, found related issue upstream material-components/material-components-web#4173

Will investigate and let ya know.

@trimox
Copy link
Owner

trimox commented Jan 29, 2019

Planning to remove touchstart from select, at least until resolved upstream.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants