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

Reposition the adopters scroller #102

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SergioCasCeb
Copy link
Contributor

@SergioCasCeb SergioCasCeb commented Nov 4, 2024

To provide a preview of where the adopters' scroller might fit best, I have temporarily added it in three different sections:

  • Header
  • About/Value Proposition section
  • Original placement (for comparison)

Personally, I lean towards placing it in the header, as it quickly builds trust with users. However, the original location also works well contextually.

Once a final placement is decided, I will clean up the code and remove the unwanted ones.

Copy link

netlify bot commented Nov 4, 2024

Deploy Preview for thingweb ready!

Name Link
🔨 Latest commit e1ec4ed
🔍 Latest deploy log https://app.netlify.com/sites/thingweb/deploys/6728250e55e57d00087c3239
😎 Deploy Preview https://deploy-preview-102--thingweb.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@danielpeintner
Copy link
Member

I like the "header" version also.
Anyhow, given that the numbers (Discord, GitHub and Adopters) is mention in the end I still think it is best there...

@egekorkan
Copy link
Member

I like the header more than the about. Not sure about the original vs header. @relu91 @JKRhb any opinion?

@relu91
Copy link
Member

relu91 commented Nov 4, 2024

+1 for the header version, I agree with the feedback that we collected. Works good on mobile too.

@JKRhb
Copy link
Member

JKRhb commented Nov 5, 2024

I also like the header version better than the about version :) I was wondering, though, whether there should be a small tagline indicating that the logos belong to adopters, to give a bit more context.

@SergioCasCeb
Copy link
Contributor Author

So far, the header version might have the edge a bit. How should we proceed from here?

@egekorkan
Copy link
Member

So far, the header version might have the edge a bit. How should we proceed from here?

Header version it is. I agree with Jan though. It would be nice to indicate "Adopted by..." before that.

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.

5 participants