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

feat(limitTo): extend the filter to take a beginning index argument #10899

Closed
wants to merge 1 commit into from
Closed

feat(limitTo): extend the filter to take a beginning index argument #10899

wants to merge 1 commit into from

Conversation

tameraydin
Copy link
Contributor

Extend the limitTo filter to take an optional argument for beginning index. It provides a slice-alike functionality to manipulate the input.

Closes #5355

Extend the limitTo filter to take an optional argument for beginning index. It provides a slice-alike functionality to manipulate the input.

Closes #5355
@gkalpak
Copy link
Member

gkalpak commented Jan 29, 2015

Now that there is an offset parameter, it might make sense to disallow negative limit, as it leads to weird semantics (and code).

@gkalpak
Copy link
Member

gkalpak commented Jan 29, 2015

But it is going to be a breaking change :(

@pkozlowski-opensource
Copy link
Member

This PR seems good after a quick look - that is - it does what it announces. Having said this the resulting API is a bit odd... but I guess this is the best we can do without API breaking changes...

Let's reach out to the community to see what other think...

@revolunet
Copy link
Contributor

very cool thanks

@pkozlowski-opensource pkozlowski-opensource self-assigned this Jan 29, 2015
netman92 pushed a commit to netman92/angular.js that referenced this pull request Aug 8, 2015
Extend the limitTo filter to take an optional argument for beginning index.
It provides a slice-alike functionality to manipulate the input.

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

Successfully merging this pull request may close these issues.

limitTo filter should have a left and right bound.
6 participants