Glad
@agutierrezb started it, now the designer in
me feel like less of an asshole for my own feedback.
I would echo most of your points I think this is a great move forward, but the yellow color may be a bit overused. That said with these kind of sites which are essentially site builders you are often tied into these colors for all CTA / actionable items. I've run into this issue before with template based sites and it's always hard to adjust for. I think the warm yellow y'all are using is the one I suggested which would make this my fault lol. I think it is just a bit loud for the amount it's being used. Perhaps if this much of the color is needed a yellow that still warm, but less orange (somewhere in the #fccc00 range?) would work better, although the visibility against white would be slightly reduced.
Agree on darkening the image over the top. I think the choice of logo works well for the site (really love that ampersand
@Alexander ) but the visibility against the header image is poor. I think if the image was darkened you could ditch that light SVG stroke which shouldn't be needed, and will render poorly on low DPI screens. Total subjective POV, but I would also change the words to white at the same time. The white on black will be much higher contrast which will be better for any visitors with visual disabilities and more pleasing to the eye. See unsolicited mockup below!
View attachment 1247