Can you point us at where in the UI these are used, exactly? Just want to make sure we're fixing the right thing.

I can't think of where these are used without text or tooltip.

I think it's maybe in the PHUIStatusView here?

I think we use for "rejected", and for "rejected older diff".

Likewise, we use for "accepted" and for "accepted older diff".

They have tooltips and the distinction isn't particularly important, but maybe an easy fix is to just use and for the "older" variants (without the background)?

They're used in a diff. Red means someone has rejected my most recent diff. Black means they rejected an older version but haven't seen my newest revision.

@chad I can differentiate by hovering with a mouse over the icon. But that is cumbersome and undesirable.

Yeah that's all I could come up with as a possibility too.

@epriestley Inverting the "old" (greyscale) icons would resolve my issue.

for current and for previous?

current reject
previous reject
current accept
previous accept

At least circle and circle-o are similar enough that it doesn't feel out of place.

Ah, sure, that makes sense.