Staff color applies to quoted message


#1

A post was written by staff member. The post is a response to another post (by non-staff member) and has the “in response to X” symbol at the top right corner. (As it happens, the message is the one right above it.) Clicking the symbol opens up the message again on top of the message (showing the message to which the post is a reply to).

The post by staff member had staff color added. The color applied (bug!) to the replied message as well.

Screenshot:


(Jeff Atwood) #2

This might be fixed with @awesomerobot’s latest change?


(Jeff Atwood) #6

Wait this isn’t a proper test because you are staff. You need to reply as a regular user.


(Jeff Atwood) #8

OK now we have repro @awesomerobot not sure if fixable in CSS without contortions…


(Sam Saffron) #9

I think it is easy enough to apply a CSS fix for this.

The reverse direction is hard though cause embedded posts now don’t have the correct class applied. (which is a seperate bug)


(Kris) #10

Just pushed a quick fix by adding additional specificity.

For future reference:

  • Selecting the nested replies: easy, because there’s .embedded-posts wrapping them
  • Selecting the parent post’s cooked content: easy, because it’s wrapped by .regular
  • Selecting the parent post’s avatar/meta: a little annoying, because there’s nothing distinguishing them from the .embedded-posts avatar/meta. At some point we should either expand the .regular wrapper to include the parent’s avatar/meta and cooked content, or just add a .regular class to them.

(Jeff Atwood) #11

Does not seem to be working yet @sam are builds not being pushed out?


(Sam Saffron) #12

Yeah our build was broken, I amended our internal process to remind us once an hour if the build is broken.


(Sam Saffron) #13