From 4917ddb3c6917d43cdf86aa5a5e196d98a1f1103 Mon Sep 17 00:00:00 2001 From: Robin <74519799+Robin5605@users.noreply.github.com> Date: Tue, 14 May 2024 12:59:18 -0500 Subject: Improve cancel staff infraction button colours and text (#2980) * Change color of cancel button from green to gray * Change "confirm" label to "ban" * Make docstring for confirm callback more accurate Co-authored-by: Vivek Ashokkumar * Rename "InfractionConfirmationView" to "BanConfirmationView" --------- Co-authored-by: Vivek Ashokkumar --- bot/exts/moderation/infraction/_utils.py | 4 ++-- bot/exts/moderation/infraction/_views.py | 8 ++++---- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/bot/exts/moderation/infraction/_utils.py b/bot/exts/moderation/infraction/_utils.py index f306ede02..b1007546a 100644 --- a/bot/exts/moderation/infraction/_utils.py +++ b/bot/exts/moderation/infraction/_utils.py @@ -12,7 +12,7 @@ import bot from bot.constants import Categories, Channels, Colours, Icons, MODERATION_ROLES, STAFF_PARTNERS_COMMUNITY_ROLES from bot.converters import DurationOrExpiry, MemberOrUser from bot.errors import InvalidInfractedUserError -from bot.exts.moderation.infraction._views import InfractionConfirmationView +from bot.exts.moderation.infraction._views import BanConfirmationView from bot.log import get_logger from bot.utils import time from bot.utils.channel import is_in_category, is_mod_channel @@ -339,7 +339,7 @@ async def confirm_elevated_user_ban(ctx: Context, user: MemberOrUser) -> bool: if not isinstance(user, Member) or not any(role.id in STAFF_PARTNERS_COMMUNITY_ROLES for role in user.roles): return True - confirmation_view = InfractionConfirmationView( + confirmation_view = BanConfirmationView( allowed_users=(ctx.author.id,), allowed_roles=MODERATION_ROLES, timeout=10, diff --git a/bot/exts/moderation/infraction/_views.py b/bot/exts/moderation/infraction/_views.py index 6215b2b6e..26287d6e6 100644 --- a/bot/exts/moderation/infraction/_views.py +++ b/bot/exts/moderation/infraction/_views.py @@ -6,21 +6,21 @@ from discord.ui import Button from pydis_core.utils import interactions -class InfractionConfirmationView(interactions.ViewWithUserAndRoleCheck): +class BanConfirmationView(interactions.ViewWithUserAndRoleCheck): """A confirmation view to be sent before issuing potentially suspect infractions.""" def __init__(self, *args: Any, **kwargs: Any) -> None: super().__init__(*args, **kwargs) self.confirmed = False - @discord.ui.button(label="Confirm", style=ButtonStyle.red) + @discord.ui.button(label="Ban", style=ButtonStyle.red) async def confirm(self, interaction: Interaction, button: Button) -> None: - """Callback coroutine that is called when the "confirm" button is pressed.""" + """Callback coroutine that is called when the "Ban" button is pressed.""" self.confirmed = True await interaction.response.defer() self.stop() - @discord.ui.button(label="Cancel", style=ButtonStyle.green) + @discord.ui.button(label="Cancel", style=ButtonStyle.gray) async def cancel(self, interaction: Interaction, button: Button) -> None: """Callback coroutine that is called when the "cancel" button is pressed.""" await interaction.response.send_message("Cancelled infraction.") -- cgit v1.2.3