Apply suggestions from code review

These are good suggestions

Co-authored-by: shiftinv <[email protected]>
Signed-off-by: Strix <[email protected]>

Author: Strixen

guide/docs/popular-topics/reactions.mdx

@@ -18,9 +18,9 @@ In this guide we will be providing an example using the <DocsLink reference="dis
 -   Removing reactions that are not owned by the bot requires <DocsLink reference="disnake.Intents.reactions">Intents.reactions</DocsLink> to be set
 -   Therefore <DocsLink reference="disnake.Intents.messages">Intents.messages</DocsLink> is indirectly required if you want to manipulate reactions
 -   A message can have a maximum of 20 unique reactions on it at one time.
--   Reactions are inherently linked to emojis, and your bot will not have access to resend all emojis used by discord users
--   Dealing with reactions result in a fair amount of extra api-calls, meaning it can have rate-limit implications on deployment scale.
--   Using Reactions as a UX interface was never a inteded behavior, and is ultimatly inferior to the newer component style interface
+-   Reactions are inherently linked to emojis, and your bot will not have access to resend all emojis used by Discord users.
+-   Dealing with reactions results in a fair amount of extra API calls, meaning it can have rate-limit implications on deployment scale.
+-   Using Reactions as a UX interface was never a intended behavior, and is ultimately inferior to the newer component style interface.
 :::
 
 <DiscordMessages>
@@ -61,27 +61,25 @@ PartialEmoji: are most often custom emojis too, but will usually represent custo
 Strings: are normally returned when Unicode CodePoints are used. These are the standard emojis most are familiar with (✅🎮💛💫)  
 but these can also come as a PartialEmoji
 
-There is also a small write up about this [`here`](//faq/general.mdx#how-can-i-add-a-reaction-to-a-message)
+There is also a small write up about this [here](../faq/general.mdx#how-can-i-add-a-reaction-to-a-message).
 
 :::note
-The examples are only meant to demonstrate how disnake interacts with Reactions, and should probably not be copied verbatim.  
-These examples are not intended for cogs, but can easily be adapted to run inside one. See: [**Creating cogs/extensions**](//getting-started/using-cogs.mdx)  
-Some examples are also available in the [`DisnakeDev`](https://github.com/DisnakeDev/disnake/tree/master/examples) github repository
+The examples are only meant to demonstrate how disnake interacts with reactions, and should probably not be copied verbatim.  
+These examples are not intended for [cogs](../getting-started/using-cogs.mdx), but can easily be adapted to run inside them.  
+Some examples are also available in the [GitHub repository](https://github.com/DisnakeDev/disnake/tree/master/examples).
 :::
 
 ### Example using on_reaction events
 
 There are a few reaction related events we can listen/subscribe to:
 
--   <DocsLink reference="disnake.on_raw_reaction_add">on_raw_reaction_add</DocsLink>
--   <DocsLink reference="disnake.on_raw_reaction_remove">on_raw_reaction_remove</DocsLink>
--   <DocsLink reference="disnake.on_raw_reaction_clear">on_raw_reaction_clear</DocsLink> Called when a message has all reactions
-    removed
--   <DocsLink reference="disnake.on_raw_reaction_clear_emoji">on_raw_reaction_clear_emoji</DocsLink> Called when a specific
-    reaction is removed from a message{' '}
+-   <DocsLink reference="disnake.on_raw_reaction_add">on_raw_reaction_add</DocsLink>, called when a user adds a reaction
+-   <DocsLink reference="disnake.on_raw_reaction_remove">on_raw_reaction_remove</DocsLink>, called when a user's reaction is removed
+-   <DocsLink reference="disnake.on_raw_reaction_clear">on_raw_reaction_clear</DocsLink>, called when a message has all reactions removed
+-   <DocsLink reference="disnake.on_raw_reaction_clear_emoji">on_raw_reaction_clear_emoji</DocsLink>, called when all reactions with a specific emoji are removed from a message
 
-There are non-raw equivilants, but they rely on the cache. If the message is not found in the internal cache, then the event is not called.  
-For this reason raw events are preffered, and you are only giving up on an included User/Member object that you can easily fetch if you need it.
+There are non-raw equivalents, but they rely on the cache. If the message is not found in the internal cache, then the event is not called.  
+For this reason raw events are preferred, and you are only giving up on an included User/Member object that you can easily fetch if you need it.
 
 -   More information about events can be found in the docs, [`here`](https://docs.disnake.dev/en/stable/api.html#event-reference)
 
@@ -90,9 +88,6 @@ This is generally not an issue since it contains everything we need, but its som
 Raw reaction events come a <DocsLink reference="disnake.RawReactionActionEvent">RawReactionActionEvent</DocsLink> which is called `payload` in the examples.
 
 ```python title="on_raw_reaction_add.py"
-import disnake
-
-
 @bot.listen()
 async def on_raw_reaction_add(self, payload: disnake.RawReactionActionEvent):
     # For this example we will have the bot post a message describing the event, and adding the emoji to that message as an exercise
@@ -103,25 +98,25 @@ async def on_raw_reaction_add(self, payload: disnake.RawReactionActionEvent):
     if not payload.guild_id:
         return  # guild_id is None if its a DM
 
-    # Raw event's contain few objects, so we need to grab the channel from the cache
+    # Raw events contain the channel ID, so we need to grab the channel from the cache
     event_channel = bot.get_channel(payload.channel_id)
 
     # With the channel in hand we can use it to post a new message like normal, Messageable.send() returns the message, and we need to store it
     event_response_message = await event_channel.send(
         content=f"Reaction {payload.emoji} added by: {payload.member.display_name}!"
     )
 
-    # Now using that stored message, we can add our own reaction to it, and the add_reaction() coroutine supports PartialEmojis so we're good to go
-    # One thing we need to consider is that the bot cannot access custom emojis outside servers they occupy (see caution below)
-    # Because of this we need to check if we have access to the custom_emoji.
-    # disnake.Emoji have a is_usable() function we can reference, but Partials do not so we need to check manually.
-    if payload.emoji.is_custom_emoji and not bot.get_emoji(payload.emoji.id):
-        return  # The emoji is custom, but could not be found in the cache.
+    # Now, we could add our own reaction to the message we just sent.
+    # One thing we need to consider is that the bot cannot access custom emojis from servers they're not members of (see caution below),
+    # because of this we need to check if we have access to the custom_emoji.
+    # disnake.Emoji has a `is_usable()` function we could reference, but partials do not, so we need to check manually.
+    if payload.emoji.is_custom_emoji() and not bot.get_emoji(payload.emoji.id):
+        return  # The emoji is custom, but from a guild the bot cannot access.
     await event_response_message.add_reaction(payload.emoji)
 ```
 
-Below is how the the listener above would react both for a Unicode CodePoint emoji and a custom_emoji the bot can't access  
-Notice how the **payload.emoji** resolved into **:disnake:** because the emoji is on a server not accessable to the bot
+Below is how the the listener above would react both for a unicode emoji and a custom emoji the bot can't access.  
+Notice how second emoji resolved into **:disnake:** because the emoji is on a server not accessible to the bot:
 
 <DiscordMessages>
 	<DiscordMessage profile="user">
@@ -154,11 +149,13 @@ Notice how the **payload.emoji** resolved into **:disnake:** because the emoji i
 		</div>
 	</DiscordMessage>
 	<DiscordMessage profile="bot">Reaction :disnake: added by: AbhigyanTrips!</DiscordMessage>
-</DiscordMessages>{' '}
+</DiscordMessages>
+
+<br/>
 
 :::caution
 We can only use custom emojis from servers the bot has joined, but we can use them interchangably on those servers.  
-Bots can make <DocsLink reference="disnake.ui.Button">buttons</DocsLink> using emojis from outside servers they occupy, this may or may not be intended behaviour from Discord and should not be relied on.  
+Bots can make <DocsLink reference="disnake.ui.Button">buttons</DocsLink> using emojis from servers they're not members of, this may or may not be intended behaviour by Discord and should not be relied on.  
 :::
 
 Here's a few ways you could filter on reactions to do various things
@@ -277,12 +274,9 @@ async def list_reactions(self, inter: disnake.MessageCommandInteraction):
         reaction_list
     ):  # We then loop through the reactions and use enumerate for indexing
         response_string += f"{index+1}. {reaction.emoji} - {reaction.count}\n"  # Using f-strings we format the list how we like
-    if (
-        not response_string
-    ):  # If the message has no reactions, response_string will be "" which evaluates as False
-        response_string = "No Reactions found"
 
-    await inter.response.send_message(response_string)
+    # If the message has no reactions, response_string will be "" which evaluates to False
+    await inter.response.send_message(response_string or "No Reactions found")
 
     # As with the previous examples, we can add reactions too