Remove pagination links if not required

Pagination is not displayed at the bottom of the page if there is no data available.

Thanks for opening this issue! Good find.

I think the check for 0 should occur in Pagination.svelte itself.

I'm not sure if there's a way to have logic like this

{#if count === 0}
  return
{/if}

or if the whole nav element has to be wrapped in

{#if count > 0 }
...
{/if}

But either way, it will keep all of the logic in the same file, and will also mean if we ever reuse Pagination we won't have to worry about copying the count check.

as per #5 added logic in pagination.svelte .

i will review this

Applied MR It works fine.
Adding Before and After screenshots also.
Moving it to RTBC.

Added check for page count.

Reviewed last MR and it LGTM. But because tests failed marking it as Needs Work.

If we do want to hide the pager for a single page of results, we'll need this one change. But I'm not 100% sure we do want that. I'll leave it to Chris or Ben to decide.

I vote to remove it as it has no use and can be confusing

Should we change check from {#if count > 0 && pageCount > 0}to {#if pageCount > 0}.
I think {#if pageCount > 0} will also hide the pagination if no data available as well as if there is only one page.

Changed {#if count > 0 && pageCount > 0} to {#if pageCount > 0}..

Great teamwork, thanks @narendraR for the suggestion about hiding the pager even further.