Repository: Cog-Creators/Red-DiscordBot

Issue:
Remove the colors in Installed Cogs list
### What component of Red (cog, command, API) would you like to see improvements on?

cog list

### Describe the enhancement you're suggesting.

To me, it is very confusing that "Installed" is red and "Uninstalled" is green. I would suggest removing the syntax highlighting from this menu altogether.

Example screenshot:

![image](https://user-images.githubusercontent.com/18037011/227386917-1170b7ca-0c5c-4a2a-8895-f6969c8c7d4b.png)

This implies to me that the things in red are things that I do not have available to me. But, maybe I'm calling up this list to see "what's installed and ready to use" as opposed to "what's ready to be installed." In that case, the color-coding is completely backwards.

In fact, even when I am pulling this cog up because I want to install something, I get confused, because to me green should be "already installed and good to go."

### Anything else?

1. This suggestion applies to every similar interface where the diff color coding is used but the POV could be considered in either direction, "what's available to use" or "what's available to install" when the command is called
2. There is a typo in this command (Available cog: -> Available cogs)


Discussion:
> There is a typo in this command (Available cog: -> Available cogs)

There is only 1 cog in that section, hence "cog" not "cogs". It has dynamic plurality. 
Ah, makes sense
No styling:
![image](https://user-images.githubusercontent.com/73763472/232599819-ce48da43-00a3-43a7-899d-cf6f727cf0ca.png)

'fix':
![image](https://user-images.githubusercontent.com/73763472/232599875-b7d729a2-4836-407f-b961-49ef28f853bc.png)

'yaml':
![image](https://user-images.githubusercontent.com/73763472/232599940-78fdc687-6372-467d-ba73-fd4d85c78dd8.png)

'markdown':
![image](https://user-images.githubusercontent.com/73763472/232600535-1dc5df64-bc58-449b-8ae0-dcb7bcdc33c9.png)


Markdown does look pretty nice. But if you feel strongly another way, my opinion on this isn't that important. I mostly just wanted a change from diff.

Thanks a lot for making this change!!
I changed it to markdown
I think Yaml provides a good contrast between package name and cog description, as well as a diff colour for the +- symbols which was the goal of the diff code block here
I do like yaml in this case
I'd go with Markdown, YAML is more likely to get problematic with some cog descriptions and it's IMO a bit too colorful. We might want to change the style of this entirely in the future but for now I'd like something that doesn't look as jarring as the `diff` style.

@rramboer are you still interested in making a PR for this? It seems that your PR got closed after you removed your fork.
I can make a PR. Give me 48 hours to get back home.

Resolving patch:
diff --git a/redbot/cogs/downloader/downloader.py b/redbot/cogs/downloader/downloader.py
--- a/redbot/cogs/downloader/downloader.py
+++ b/redbot/cogs/downloader/downloader.py
@@ -625,14 +625,14 @@ async def _repo_list(self, ctx: commands.Context) -> None:
             joined = _("There are no repos installed.")
         else:
             if len(repos) > 1:
-                joined = _("Installed Repos:\n\n")
+                joined = _("# Installed Repos\n")
             else:
-                joined = _("Installed Repo:\n\n")
+                joined = _("# Installed Repo\n")
             for repo in sorted_repos:
                 joined += "+ {}: {}\n".format(repo.name, repo.short or "")
 
         for page in pagify(joined, ["\n"], shorten_by=16):
-            await ctx.send(box(page.lstrip(" "), lang="diff"))
+            await ctx.send(box(page.lstrip(" "), lang="markdown"))
 
     @repo.command(name="info")
     async def _repo_info(self, ctx: commands.Context, repo: Repo) -> None:
@@ -1333,9 +1333,9 @@ async def _cog_list(self, ctx: commands.Context, repo: Repo) -> None:
         )
 
         if len(installed_cogs_in_repo) > 1:
-            installed_str = _("Installed Cogs:\n{text}").format(text=installed_str)
+            installed_str = _("# Installed Cogs\n{text}").format(text=installed_str)
         elif installed_cogs_in_repo:
-            installed_str = _("Installed Cog:\n{text}").format(text=installed_str)
+            installed_str = _("# Installed Cog\n{text}").format(text=installed_str)
 
         available_cogs = [
             cog for cog in repo.available_cogs if not (cog.hidden or cog in installed_cogs_in_repo)
@@ -1346,14 +1346,14 @@ async def _cog_list(self, ctx: commands.Context, repo: Repo) -> None:
         )
 
         if not available_str:
-            cogs = _("Available Cogs:\nNo cogs are available.")
+            cogs = _("> Available Cogs\nNo cogs are available.")
         elif len(available_cogs) > 1:
-            cogs = _("Available Cogs:\n{text}").format(text=available_str)
+            cogs = _("> Available Cogs\n{text}").format(text=available_str)
         else:
-            cogs = _("Available Cog:\n{text}").format(text=available_str)
+            cogs = _("> Available Cog\n{text}").format(text=available_str)
         cogs = cogs + "\n\n" + installed_str
         for page in pagify(cogs, ["\n"], shorten_by=16):
-            await ctx.send(box(page.lstrip(" "), lang="diff"))
+            await ctx.send(box(page.lstrip(" "), lang="markdown"))
 
     @cog.command(name="info", usage="<repo> <cog>")
     async def _cog_info(self, ctx: commands.Context, repo: Repo, cog_name: str) -> None:
diff --git a/redbot/cogs/trivia/session.py b/redbot/cogs/trivia/session.py
--- a/redbot/cogs/trivia/session.py
+++ b/redbot/cogs/trivia/session.py
@@ -279,10 +279,10 @@ async def end_game(self):
 
     async def send_table(self):
         """Send a table of scores to the session's channel."""
-        table = "+ Results: \n\n"
+        table = "Results:\n\n"
         for user, score in self.scores.most_common():
             table += "+ {}\t{}\n".format(user, score)
-        await self.ctx.send(box(table, lang="diff"))
+        await self.ctx.send(box(table, lang="markdown"))
 
     def stop(self):
         """Stop the trivia session, without showing scores."""


Test patch: