Add channel ID to message deletion logs

[TizzySaurus]

Closes #620.

The channel ID is now shown in message deletion logs.

Zoomed in on one log message:

Fullscreen page:

[netlify]

✔️ Deploy Preview for pydis-static ready!

🔨 Explore the source changes: 1b251d4

🔍 Inspect the deploy log: https://app.netlify.com/sites/pydis-static/deploys/62181485541c2000085be251

😎 Browse the preview: https://deploy-preview-682--pydis-static.netlify.app

[coveralls]

Coverage remained the same at 100.0% when pulling 1b251d4 on add-channel_id-to-clean-logs into 9d1ea16 on main.

[jchristgit]

Thanks!

[minalike]

Tested and it works fine and renders as you'd expect, but semantically wouldn't it make sense to not have line break within a span element, and rather split it into 2 spans?

One for the user ID, a line break, and a separate one for the line after with the timestamp and channel ID.

I would suggest:

            <div class="discord-message-header">
                <span class="discord-username" style="color: {{ message.author.top_role.colour | hex_colour }}">
                      {{ message.author }}
                </span>
                <span class="discord-message-metadata has-text-grey">
                    User ID: {{ message.author.id }}
                </span>
                <br>
                <span class="discord-message-metadata has-text-grey">
                    {{ message.timestamp }} (Channel ID: {{ message.channel_id }})
                </span>
            </div>

and in logs.css adjust the margin so it's only present on the right:

.discord-message-metadata {
    font-size: 0.75rem;
    font-weight: 400;
    margin: 0 .3rem 0 0;
}

But I realize this may be overkill, and there may also be a better way of doing this.

[MarkKoz]

On the principle of semantics, I agree with mina. A span is generally not seen as something that is multi-line. However, in practice, I am not so sure that it matters. If I were the one to implement this, I would likely follow mina's suggestion. However, I am overall indifferent, so I would leave this decision to the author's discretion.

[jchristgit]

Thanks!