feat: option to hide paginator buttons on stop/timeout #1660
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request Type
Description
Adds a bool attribute
hide_buttons_on_stop
toPaginator
ininteractions.ext.paginators
. If set toTrue
, the paginator will hide/clear all of its components on (1) a timeout or (2) a call toPaginator.stop()
. Otherwise, the components are disabled, which is the current (and with this PR, default) behavior of the paginator.Changes
hide_buttons_on_stop
toPaginator
Paginator.create_components()
[]
when both the method argdisable
and the new attribute areTrue
.Related Issues
Test Scenarios
Default (disable)
The buttons are disabled (greyed out) after a timeout of 20 seconds.
Timeout
The buttons are cleared after a timeout of 20 seconds.
Stop
The buttons are cleared after 8 seconds (due to
asyncio.sleep()
) since sending.Python Compatibility
3.10.x
3.11.x
Checklist
pre-commit
code linter over all edited files