Karl Stolley

All right. Please re-create all of your screenshots with the original PR above. The very first one I checked is all wrong: 

Ah, that's right. Okay, so how about this plan: your `copy_to_clipboard_svg.hbs` should get the `` portion of this in its pure form, with any additional variables (IDs, classes) passable as...

Something is not right. I'm still seeing this on Firefox/Mac:  And Chrome/Mac: 

These colors and other values should be expressed as CSS variables; let's not go the route of adding more lines to `dark_theme.css`. To do this, you'll likely need to make...

> @karlstolley If that's the case for the table border color, shouldn't the other parts of the table also be refactored in the same way? The obvious one I noticed...

A few code-organization nits. I'm not certain we won't see some regressions from striking those table-oriented element selectors in dark mode, but the fixes for those would be to use...

This looks like a reasonable set of changes to me, minus one little nit about a variable name.

@alya yes, I'll tackle this one tomorrow (January 25)

@ecxtacy have you updated screenshots since adjusting where the bot icon sits relative to the text? The screenshots as presented do show quite a bit of variation. In terms of...

@ecxtacy can you resolved the merge conflicts here and @-message me so I can do another review pass on this?