Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

fix(tabs): fix long tab content not scrolling #10586

Merged
merged 1 commit into from
Apr 14, 2017

Conversation

jelbourn
Copy link
Member

No description provided.

@googlebot googlebot added the cla: yes PR author has signed Google's CLA: https://opensource.google.com/docs/cla/ label Apr 13, 2017
Copy link
Member

@devversion devversion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Might be worth documenting it in the JSDoc as well?


function postLink(scope, element, attr) {
var touchAction = attr['mdSwipeTouchAction'] || 'none';
element.css('touch-action', touchAction);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't we just put that in a single line?

element.css('touch-action', attr.mdSwipeTouchAction || 'none');

@jelbourn jelbourn force-pushed the tabs-swipe-touch-action branch from 5c3c15f to 0dc542b Compare April 13, 2017 19:50
@jelbourn
Copy link
Member Author

Not sure if I want to advertise the API yet

@devversion
Copy link
Member

Hm no idea. It's a pretty simple feature. But anyways, LGTM (there is no label on M1)

@jelbourn jelbourn merged commit 562b1c9 into angular:master Apr 14, 2017
GulajavaMinistudio added a commit to GulajavaMinistudio/material that referenced this pull request Apr 15, 2017
fix(tabs): fix long tab content not scrolling (angular#10586)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes PR author has signed Google's CLA: https://opensource.google.com/docs/cla/
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants