dombizita commented on PR #5658: URL: https://github.com/apache/ozone/pull/5658#issuecomment-1881212391
> Hello @smitajoshi12, thanks for the info, I checked locally and the limit param is being sent properly. For the pagination as I understood from the call, No Records is the placeholder equivalent for `<<` or the prev-page button. > > But I was suggesting that, we can instead start page 1 from there and completely remove previous page button from first page. Then onwards on page 2 it will be `<< 2 >>` and so on. > > So: `1 >>` then `<< 2 >>`, `<< 3 >>`, `<< 4` This is assuming 4th page is the last page. I agree with @devabhishekpal, I wouldn't show the `No Records`, rather just nothing. Just like in the example he wrote in the last sentence. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
