[www/showcase] Placement of the "Visit Site" button on modals

[berraknil]

The current placement of the "visit site" button makes it harder to read (because it is on top of the image)

This might be a better place with the rest of the buttons/links for Source etc?

@fk what do you think?

[fk]

👍 I agree—that part of the design wasn't very well planned!

A community member brought this up a while ago in #6098, too. There I followed up with a quick mockup of how a "realigned" header could look: #6098 (comment)

The scope of the aforementioned issue is the "Site Showcase", however we also use the same design to display the detail page of a "Starter" in the "Starter Showcase". The "Starter" detail header has slightly different requirements ("Deploy to Netlify" button, and potentially a couple more) and I still need to consolidate those.

[fk]

Hey Berrak, forgot to update you on this: I posted a first stab at trying to consolidate header designs of site and starter showcase in #6710 (comment). Would love to hear your thoughts if you have the time!

[davad]

I just submitted a PR (#9320) that moves the links above the screenshot. I didn't implement @fk's mockup, just moved the existing components.

[KyleAMathews]

@davad thanks! @fk do you think this is "good enough" now?

[fk]

Late to the party, yep this is good to go … and already merged I see :/!

Thanks @davad, that brings us much closer to #6710 (comment) and in line with what we do for the starter showcase detail view! 🙏 🤗

Closed via #9320. ✌️