We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
node.navigate($.ui.keyCode.DOWN, false);
should return promise, per documentation, but since 2.31 returns undefined.
I tracked it down to the fact, that in 2.30.2 and earlier, there was one return with:
return res || _getResolvedPromise()
which guaranteed that there's always a promise.
But in 2.31, there's multiple returns without "guard":
return node.setFocus();
.setFocus() returns undefined, but this is now "unguarded", so the client code gets undefined and not promise.
The text was updated successfully, but these errors were encountered:
Good catch. This method was refactored to handle the new viewport feature and introduced that regression.
Sorry, something went wrong.
e8ab014
No branches or pull requests
should return promise, per documentation, but since 2.31 returns undefined.
I tracked it down to the fact, that in 2.30.2 and earlier, there was one return with:
which guaranteed that there's always a promise.
But in 2.31, there's multiple returns without "guard":
.setFocus() returns undefined, but this is now "unguarded", so the client code gets undefined and not promise.
The text was updated successfully, but these errors were encountered: