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

feat(virtualRepeat): Add md-auto-shrink and md-auto-shrink-min #3536

Closed

Conversation

kseamon
Copy link
Contributor

@kseamon kseamon commented Jul 1, 2015

These attributes instruct the container to shrink to fit when the total size of repeated items is smaller than the available space. This will unblock integration with md-autocomplete.

scope.$apply();
expect(container[0].offsetHeight).toBe(2 * ITEM_SIZE);
});

Copy link
Member

Choose a reason for hiding this comment

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

No test with md-horizontal and md-auto-shrink-min ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added one. Does not test any new code paths though.

@jelbourn
Copy link
Member

jelbourn commented Jul 1, 2015

LGTM

@kseamon kseamon force-pushed the virtual-repeat-container-auto-shrink branch from 4fb4025 to efde978 Compare July 1, 2015 18:44
@kseamon kseamon force-pushed the virtual-repeat-container-auto-shrink branch from efde978 to 254d7fd Compare July 1, 2015 18:52
@kseamon
Copy link
Contributor Author

kseamon commented Jul 1, 2015

Pushed an update with the comments addressed.

@kseamon kseamon closed this in 79dcbf7 Jul 1, 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 this pull request may close these issues.

3 participants