Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removes hrefs from homepage logos #3331

Closed
wants to merge 1 commit into from
Closed

Removes hrefs from homepage logos #3331

wants to merge 1 commit into from

Conversation

benjaminhoffman
Copy link
Contributor

I absolutely love the idea of using logos as social proof but not super keen on sending our hard-earned traffic immediately away, especially since these are above the fold on the homepage.

This PR may be controversial so feel free to close & not merge. My theory and experience shows that having partner logos that are clickable does lead to a loss of at least some traffic loss.

@ghost ghost assigned benjaminhoffman Dec 24, 2017
@ghost ghost added the review label Dec 24, 2017
@gatsbybot
Copy link
Collaborator

Deploy preview ready!

Built with commit 2c36c17

https://deploy-preview-3331--gatsbygram.netlify.com

@KyleAMathews
Copy link
Contributor

I'd love there to be a showcase page on the website. How do you feel about linking there?

@KyleAMathews
Copy link
Contributor

When it's made of course :-)

@benjaminhoffman
Copy link
Contributor Author

yeah much better! keeping traffic on the site is the end goal. we can circle back on this once that's live. will close this for now I guess. also, Segment's blog is what uses gatsby yet we link to its homepage.

@ghost ghost removed the review label Dec 24, 2017
@fk
Copy link
Contributor

fk commented Dec 28, 2017

Good points @benjaminhoffman. 🙏
+1 based on personal experience.
@KyleAMathews might have some analytics numbers on that? ;-)

also, Segment's blog is what uses gatsby yet we link to its homepage.

Uhhh true. :-/
That's misleading – will fix later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants