Skip to content

Added custom attribute data-instantActivate #2394

Closed
mirko77 wants to merge 4 commits intoionic-team:masterfrom
mirko77:master
Closed

Added custom attribute data-instantActivate #2394
mirko77 wants to merge 4 commits intoionic-team:masterfrom
mirko77:master

Conversation

@mirko77
Copy link
Copy Markdown

@mirko77 mirko77 commented Oct 14, 2014

Added custom attribute data-instantActivate to remove 80ms activation delay when an element is touched (down to 1ms)

Forum thread here http://forum.ionicframework.com/t/no-visual-feedback-on-quick-taps/7280/10

… delay when an element is touched (down to 1ms)
@mhartington
Copy link
Copy Markdown
Contributor

Can you make the changes in the correct file?

https://github.com/driftyco/ionic/blob/master/js/utils/activator.js

@mirko77
Copy link
Copy Markdown
Author

mirko77 commented Oct 14, 2014

Oh sorry, done

@mhartington
Copy link
Copy Markdown
Contributor

Awesome, I'll let @adamdbradley or @perrygovier look this over

@PureSpider
Copy link
Copy Markdown

Any news regarding this?

@wreiske
Copy link
Copy Markdown
Contributor

wreiske commented Nov 19, 2014

+1!

1 similar comment
@cyprusglobe
Copy link
Copy Markdown

+1!

@jrowny
Copy link
Copy Markdown

jrowny commented Nov 24, 2014

+1

3 similar comments
@tairezzzz
Copy link
Copy Markdown

+1

@vuk-nikolic
Copy link
Copy Markdown

+1

@ecunado
Copy link
Copy Markdown

ecunado commented Dec 4, 2014

+1

@adamdbradley
Copy link
Copy Markdown
Contributor

Updates have been made to the nightly builds to improve visual feedback. Closing this PR because the new system checks if its actively scrolling or not, which was the 80ms delay's intent.

@PureSpider
Copy link
Copy Markdown

It's commit d1c92da for anyone wondering.

@tairezzzz
Copy link
Copy Markdown

thanks

2014-12-07 23:13 GMT+02:00 PureSpider notifications@github.com:

It's commit d1c92da
d1c92da
for anyone wondering.


Reply to this email directly or view it on GitHub
#2394 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants