My personal take is, "great genes" is an eugenic expression that has passed into popular vocabulary and gets used without thought beyond "oh that person probably had hot parents." Which makes it not great but not radically Nazi usually AND also catnip for Nazis.

So there are two different levels of should have known better here: 1) they should have known better in the same sense that we all should be better educated on words that have an ableist/eugenic history (like d*mb or l*me) and drop them from our vocabulary, and

2) they should have known better because we are living in a moment of explicit white supremacist revivalism and explicit white natalism and you just do not talk in broad terms about how great a white lady's "genes" are in that context.

As far as (1) goes, we realistically just are not at a place where I expect most people to have a sense of the eugenic/ableist history of a lot of these still-popular words and expressions. It doesn't make it okay but it also means they aren't automatically INTENTIONAL political agenda markers.

(2) to me though feels pretty obvious to anyone with any kind of awareness of this political moment, which is something marketing execs really ought to be attuned to. It makes me a lot more suspicious of who they might be winking at, just a little bit.