-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pagingOptions.currentPage doesn't get set properly with Angular >1.1.1. #598
Comments
👍 Also happens w/ ng-grid 2.0.7 w/ ng 1.2.0rc1 😨 This appears to be some kind of data-binding issue with the type="number" and ng parsing it as a string? I changed the template to use input="text" and everything seems to work. |
+1 |
+1 |
Also experiencing this issue with 1.2.0rc1, easily replicatable by changing the angular version referenced in the serverside paging plunkr http://plnkr.co/edit/50vJrs?p=preview |
Just spent some time looking into this - for an easy fix just remove the |
My solution was to change the input type to text from number as validation for number doesn't appear to work properly. On Sep 3, 2013, at 10:58 AM, Christopher Najewicz notifications@github.com wrote:
|
@geoffreytran yeah, the |
fixed with above PR |
$scope.yourGridOptions.paginationCurrentPage = 1; --> will reset the pagenumber of angular ui-grid :) |
Using Ng-grid 2.0.7 pagingOptions.currentPage doesn't get set properly with Angular >=1.1.3 (The version provided by ng-grid in the lib folder).
It does work when reverting back to 1.0.2 which is the version defined in the plnkr provided on the ng-grid documentation page. The regression seems to have been introduced in 1.1.3 (1.1.1 works fine).
An example that illustrates the issue:
http://plnkr.co/edit/jxdukO?p=preview
The input field used to display the current page shows no initial value, using the paging buttons does not change the value as expected even after inputing a value manually. Also the validation doesn't seem to work as the input gets marked as invalid for valid numbers.
The text was updated successfully, but these errors were encountered: