fix(module:table): updated the condition for displaying nz-pagination…

[gitkhartmann]

… when nzTotal > 0

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Documentation content changes
• Application (the showcase website) / infrastructure changes
• Other... Please describe:

What is the current behavior?

In order to hide the nz-pagination, one more condition is not taken into account: when nzTotal > 0

Issue Number: N/A

What is the new behavior?

now nz-pagination will be displayed when nzTotal > 0

Does this PR introduce a breaking change?

• Yes
• No

Other information

[zorro-bot]

This preview will be available after the AzureCI is passed.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.50%. Comparing base (9dd14ae) to head (465ed90).
Report is 32 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #8628   +/-   ##
=======================================
  Coverage   91.50%   91.50%           
=======================================
  Files         534      534           
  Lines       18425    18425           
  Branches     2905     2905           
=======================================
  Hits        16860    16860           
  Misses       1244     1244           
  Partials      321      321           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[OriginRing]

Why do you need to show pagination when there is no data? 🤔

[gitkhartmann]

Why do you need to show pagination when there is no data? 🤔

Hi, my friend.

We currently have an issue with pagination: if the user is on page 2 and the backend returns empty data, but the total number of records (nzTotal) is greater than zero, the pagination disappears. As a result, the user cannot navigate back to the first page where data might be present. To work around this issue, the user has to clear the table filters to unlock the pagination.

This enhancement is needed so that the user can manage pagination and return to the first page if there is no data on the current page.

I have sent a video file to your email ([email protected]) where you can see exactly what this issue looks like.

[OriginRing]

LGTM

[gitkhartmann]

LGTM

HI, my friend!

The verification (License Compliance) failed with an error, can I somehow influence this verification to be completed successfully?

[Laffery]

The verification (License Compliance) failed with an error, can I somehow influence this verification to be completed successfully?

Ignore it, 'cause no npm dependency update.

[Laffery]

Please add enough unit tests for this feature :)