What have you changed?

This adds support for a text argument for the Pagination component. I've done this by adding a text argument which accepts a method that can return React.ReactNode. It takes a prop argument of all the variables that a user might want to use to generate custom text.

What are you trying to achieve?

I have a table of data which can have thousands of entries in it. Pagination is done via token.

In current usage, we are aware of what the totalCount is, which allows me to pass it in and this will all work fine.

However, we are looking at adding search functionality to this table. In this new circumstance, we will not be aware of what the total count is. Two things are true of our current API design:

1. in order to have a next button enabled, totalCount needs to be set
2. if we have totalCount set, it will appear in this text

While the wrapping interface will be a little ugly, this will allow me to "fake" the totalCount if I have a continuation token - while ensuring that this isn't shown in the text.

What other possibilities are there?

I've explored this space a bit in my previous PR, which introduced "simple" pagination for this component. Even if this is a little gross on my side, this feels the cleanest way from a this API point of view.

Other possible iterations on the text={({perPage}) => `Hey! ${perPage}`} solution could be either:

• just allow text text={`Hey! ${perPage}`} - as all of the context is technically being passed into the Pagination component, you could avoid the method. I like the fact that our current method gives us easy access to the {pageShowingRange} variable however
• allow this in addition to the callback (e.g. support text={"my text"} and text={() => "my text"}
• instead of passing {pageShowingRange} as a single variable that looks like 1-9, we could pass a {lowerItem} and {upperItem} instead (or similar), as the dash feels a bit like a presentation concern

Other potential changes here could be that text isn't a great prop name - description could work better? Happy to change if there are any better suggestions