-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Loads a new git history page when user scrolled list to the very bottom #3553
Conversation
Promise<LogResponse> log(DevMachine devMachine, Path project, Path[] fileFilter, boolean plainText); | ||
|
||
/** | ||
* Get log of commits. The result is the list of {@link Revision}, which is returned by callback in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
which is returned by callback
can't see any callback here 😸
private void fetchHistoryPage(final boolean append) { | ||
service.log(appContext.getDevMachine(), | ||
project.getLocation(), | ||
null, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
null
is passed but log
method's javadoc says nothing about it.
I think it would be nice to have documented this possibility.
@@ -87,17 +91,15 @@ | |||
Button btnDiffWithWorkTree; | |||
@UiField | |||
Button btnDiffWithPrevCommit; | |||
@UiField | |||
ScrollPanel scrollPanel; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isn't formatted
Promise<LogResponse> log(DevMachine devMachine, Path project, Path[] fileFilter, boolean plainText); | ||
|
||
/** | ||
* Get log of commits. The result is the list of {@link Revision}, which is returned by callback in | ||
* <code>onSuccess(Revision result)</code>. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Paraphrase this sentence, this api method returns a promise, it doesn't operate with callbacks.
@@ -87,17 +91,15 @@ | |||
Button btnDiffWithWorkTree; | |||
@UiField | |||
Button btnDiffWithPrevCommit; | |||
@UiField | |||
ScrollPanel scrollPanel; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apply formatter to this piece of code
@azatsarynnyy |
Build success. https://ci.codenvycorp.com/job/che-pullrequests-build/1480/ |
…om (eclipse-che#3553) CHE-3353: Pagination in loading git history
What does this PR do?
Pagination at loading git history
What issues does this PR fix or reference?
#3353
Previous behavior
All git history was loaded at one request and blocked ide.
New behavior
Loads git history by page when user scrolled list to the very bottom
Please review Che's Contributing Guide for best practices.