added contact us button and email validation #126
Conversation
|
can you share a screenshot to show how Contact us is looking in smaller size such as mobile phones |
|
I haven't added it for the mobile view yet, i missed it. |
|
This is the mobile view @kaustubhai I have also added the required spacing between the buttons in mobile view which was lacking earlier |
|
This doesnt look right |
the alignment? -> it was already that way i just added one more button |
|
The current alignment ☝️ |
|
i think we are not using the same source code files maybe, i cloned the whole code again yesterday itself and even now after your comment, but still it's showing to me that way only, ( the login button not aligned properly). |
|
@jainmudit68 yes there is bug in the bottom nav bar. |
|
@jainmudit68 contact us button is not required in mobile view it looks very wierd |
| > | ||
| {' '} | ||
| | ||
| |
There was a problem hiding this comment.
why there are so many spacing?
There was a problem hiding this comment.
the icons were looking too close in the mobile view so i increased the space between them.
Like the space between the first two icons originally was ok but others were too close to be distinguishable
| style={{ color: `${nav === 'contactus' ? 'wheat' : 'gray'}` }} | ||
| > | ||
| {' '} | ||
| |
| > | ||
| {' '} | ||
| | ||
| |
varun-singhh
added
gssoc23
3 participants
this closes issues #82 and #101 and is referenced with PR #104