-
Notifications
You must be signed in to change notification settings - Fork 27
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
Redesign Hero Section #4
Comments
@ajaynegi45 hi, please assign this to me, i will fix this as soon as i receive assignment |
Assignee: @Sky-De Hi @Sky-De , Thank you for your willingness to work on the "Redesign Hero Section" issue. I've just assigned the issue to you. You can now proceed with the task, and we look forward to seeing your improvements. If you have any questions or need further assistance during the process, please feel free to ask. |
@ajaynegi45 hi again, this issue is resolved, actually I just commented out the Hero component of the code and redesigned the search section, because it was better UX and UI practice to use this section rather than the Hero section, Gifs BEFORE and AFTER for showcasesDESKTOP--before DESKTOP--after MOBILE--before MOBILE--after If you have any questions , feel free to ask, otherwise let me know to create a PR for this issue |
@ajaynegi45 thanks for your response
Last thing i highly recommend you before i change anything let me create PR then you merge it and take a look at it as live project, after that if you were not satisfied you can simply let me know and I'll change that to your desired design |
Create pr I will check. #4 (comment) This is just for example purpose not for implementation. |
No description provided.
The text was updated successfully, but these errors were encountered: