Skip to content
This repository has been archived by the owner on Aug 29, 2023. It is now read-only.

md-tabs Does not work on mobile using touch, works on desktop #1403

Closed
dberube opened this issue Feb 5, 2015 · 11 comments
Closed

md-tabs Does not work on mobile using touch, works on desktop #1403

dberube opened this issue Feb 5, 2015 · 11 comments
Assignees
Milestone

Comments

@dberube
Copy link

dberube commented Feb 5, 2015

Nothing I do can get md-tabs to work on mobile, works fine in browser.

I can't remember what the last version I used where it works on mobile, I'd like to say it was 0.7.0-rc2...

@dberube
Copy link
Author

dberube commented Feb 5, 2015

0.7.0-rc3 is last version where md-tabs work on mobile for me.

md-list also doesn't work on mobile touch of an item with link after this version either...

@marcysutton
Copy link
Contributor

@dberube can you provide more details about your mobile platform and browser, as well as a Plunker or Codepen with your exact code? Tabs work fine on iOS 8/Safari.

Regarding links within md-list, can you file a separate issue with a Plunker or Codepen as well?

@justingrayston
Copy link
Contributor

Tab Demos at v0.7.1 - SHA cd5f91d run ok for me on Android 5.0.1 Chrome

@dberube
Copy link
Author

dberube commented Feb 6, 2015

Don't have code on me right now, but I forgot to mention that this is in an Ionic Framework app on Android, if that is relevant.

OS: Mac OS X Yosemite
Node Version: v0.10.33
Cordova CLI: 4.2.0
Ionic CLI Version: 1.3.8
Xcode version: Xcode 6.1.1 Build version 6A2008a
ios-sim version: Not installed

@BenoitClaveau
Copy link

When I selecting a tab and my finger slide down outside the tab, the ripple effect start but event is cancelled. On mobile is it difficult to click on tab because tab are small.

@mikkokam
Copy link

mikkokam commented Feb 8, 2015

@dberube - Combo Ionic + 0.7.1 will not work on mobile / touch devices or Chrome emulation of a touch device. Might be an issue with the new built-in hammer.js
See also:
#1406

@ThomasBurleson
Copy link
Contributor

@mikkokam , @dberube - Angular Material 0.7.1 no longer depends upon or uses HammerJS. Instead we use a custom $mdGesture service.

@louisdmli
Copy link

+1. Same problem for me. I'm also using Ionic Framework.

@dberube
Copy link
Author

dberube commented Feb 9, 2015

Yeah, definitely appears to be an issue when used with ionic or cordova (phonegap).

@mydearxym
Copy link

some issue as @beny78 describe

@infomofo
Copy link

The conflict is specifically with fastclick, which is bundled native with ionic and many cordova forks. I've described the issue in more detail here. It seems to affect a few of the core material-design directives.

@ThomasBurleson ThomasBurleson modified the milestone: 0.8.0 Feb 17, 2015
@robertmesserle robertmesserle modified the milestones: 0.8.0, 0.9.0 Feb 19, 2015
robertmesserle added a commit that referenced this issue Mar 17, 2015
Closes #1850
Closes #1698
Closes #1570
Closes #1564
Closes #1518
Closes #1516
Closes #1506
Closes #1505
Closes #1403
Closes #1387
Closes #1380
Closes #1261
Closes #1247
Closes #1140
@ajoslin ajoslin added the in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs label Mar 17, 2015
robertmesserle added a commit that referenced this issue Mar 18, 2015
Closes #1850
Closes #1698
Closes #1570
Closes #1564
Closes #1518
Closes #1516
Closes #1506
Closes #1505
Closes #1403
Closes #1387
Closes #1380
Closes #1261
Closes #1247
Closes #1140
robertmesserle added a commit that referenced this issue Mar 19, 2015
Closes #1850
Closes #1698
Closes #1570
Closes #1564
Closes #1518
Closes #1516
Closes #1506
Closes #1505
Closes #1403
Closes #1387
Closes #1380
Closes #1261
Closes #1247
Closes #1140
Closes #1626
robertmesserle added a commit that referenced this issue Mar 19, 2015
Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1626
Closes #1698
Closes #1850
robertmesserle added a commit that referenced this issue Mar 19, 2015
Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1626
Closes #1698
Closes #1777
Closes #1850
robertmesserle added a commit that referenced this issue Mar 19, 2015
Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1850
robertmesserle added a commit that referenced this issue Mar 19, 2015
Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1850
robertmesserle added a commit that referenced this issue Mar 20, 2015
Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
robertmesserle added a commit that referenced this issue Mar 20, 2015
Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
Closes #1986
robertmesserle added a commit that referenced this issue Mar 24, 2015
BREAKING CHANGE: Generated HTML structure has changed, so custom styles
will need to be updated to match the new HTML structure.

Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
Closes #1986
robertmesserle added a commit that referenced this issue Mar 24, 2015
BREAKING CHANGE: Generated HTML structure has changed, so custom styles
will need to be updated to match the new HTML structure.

Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
Closes #1986
robertmesserle added a commit that referenced this issue Mar 25, 2015
BREAKING CHANGE: Generated HTML structure has changed, so custom styles
will need to be updated to match the new HTML structure.

Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
Closes #1986
robertmesserle added a commit that referenced this issue Mar 25, 2015
BREAKING CHANGE: Generated HTML structure has changed, so custom styles
will need to be updated to match the new HTML structure.

Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
Closes #1986
Closes #2020
robertmesserle added a commit that referenced this issue Mar 26, 2015
BREAKING CHANGE: Generated HTML structure has changed, so custom styles
will need to be updated to match the new HTML structure.

Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
Closes #1986
Closes #2020
@ajoslin ajoslin removed in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs resolution: fixed labels Mar 26, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.