-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Homepage: visual switch for web applications section on mobile #297
Homepage: visual switch for web applications section on mobile #297
Conversation
Fixed it this way, do you have a suggestion so we don't need to change text twice. Let me know and I'll do the changes + the all contributors step. |
Super! Hmm yes, I can think of 2 approaches we can do here:
Would be great to have option 1, but only if it makes sense with the current layout. Otherwise 2 is the best option, as it's very clear. Thanks a lot for the contribution! Great to have a new contributor for CodeSandbox 😄 |
I also agree 1 would be the best option, taking a look how much impact it has. |
@CompuIves Think I nailed it. 🎉 |
That's great!! I can merge it in tomorrow, when I'm back at my MacBook. Thanks a lot! |
Sorry about the multiple contributor commits. I thought it didn't work, but apparently it did :) |
This is great, @duivvv. Thanks! Merging it in now, welcome to the contributors 😄 . |
No problem, glad I could help |
What kind of change does this PR introduce?
homepage change
changed position of Tailored for web applications intro as discussed here https://twitter.com/CompuIves/status/931253159871295488