Clueless-Community / clueless-official-website

This is Clueless official website where you can make your developer profile as well as meet new people across the world.

https://clueless-community.vercel.app/

MIT License

79 stars 139 forks source link

created lacking skills #513

Closed amdpt323 closed 1 year ago

amdpt323 commented 1 year ago

Created lacking skills section

there is one svg missing that I couldn't create myself and this is not yet mobile optimized have to add media query also I have just pushed so that I could get a review of over all and then I could improve it. Thankyou

508

Fixes #()

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ *] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ *] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
[ *] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[* ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes needed to the documentation

vercel[bot] commented 1 year ago

@amdpt323 is attempting to deploy a commit to a Personal Account owned by @Clueless-Community on Vercel.

@Clueless-Community first needs to authorize it.

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated
clueless	✅ Ready (Inspect)	Visit Preview	💬 Add your feedback	Mar 8, 2023 at 7:05AM (UTC)

amdpt323 commented 1 year ago

@Debajyoti14 please look into this pr

amdpt323 commented 1 year ago

@Debajyoti14
for the current source code im getting this as an output but idk why the deployment is showing some layout and image issue

Debajyoti14 commented 1 year ago

@amdpt323 I exactly don't know the reason. Can you try by removing the unnecessary mui components like Typography, Grid where we can use h1 or div instead? This may fix then

amdpt323 commented 1 year ago

@Debajyoti14 can you look into this

Debajyoti14 commented 1 year ago

@amdpt323 The Typography is fixed now But it's not Responsive and also the blue button need to be fixed

amdpt323 commented 1 year ago

@Debajyoti14 i hope this update fixes all issue.

Debajyoti14 commented 1 year ago

@amdpt323 Not fixed yet

amdpt323 commented 1 year ago

@Debajyoti14 last for today please check

Debajyoti14 commented 1 year ago

@amdpt323 Fix this issue and also I don't know why it's padding is not adjusted still now

Debajyoti14 commented 1 year ago

@amdpt323 It's good to be merged Fix this bottom space issue and Add some space btw button and image in mobile view Fix the Merge Conflicts too

amdpt323 commented 1 year ago

@Debajyoti14 the bottom space you are mentioning is due to the upcomingEvent section do you want me to change that ?

Debajyoti14 commented 1 year ago

@amdpt323 Then don't change it