-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
chore(bridge): change homepage hero for mobile view #405
Conversation
@Rishav1707 is attempting to deploy a commit to the taikoxyz Team on Vercel. A member of the Team first needs to authorize it. |
Please run the workflows awaiting approval... @d1onys1us @dong77 @mfinestone @RogerLamTd |
That makes sense and I agree! |
I have committed the required changes... @d1onys1us @wolfderechter @dong77 @schlusslicht Screenshots |
Hi @Rishav1707 I don't see what the difference is between this and the current homepage, aside from the horizontal line. I believe we want to put the grid pattern "behind" the text, or leave it as is. |
@dantaik @shadab-taiko Please review my changes... |
@Rishav1707 Did you see my comment? We might not need this change in the current state, and instead would either 1) keep it the same or 2) move the grid background "behind" the text. |
Thanks for giving me an opportunity to contribute. |
This PR closes the following issues :-
closes #399
Changes made :-
Screenshots
Before
After