CSEdgeOfficial / learn.csedge.courses

This is a blog for the CSEdge which consists simple articles on different tech things how to do? and what to do?
https://learn.csedge.courses
MIT License
67 stars 136 forks source link

Made site responsive #412

Closed pankhuri92 closed 1 month ago

pankhuri92 commented 3 months ago

Related Issue

The site wasn't responsive in versions from laptop-1024px Fix #392

Description

-The images were bigger than needed and moving out of screen. I reduced their scaling. -The text size were larger than needed making it shabby. Adjusted the size. -I changed the paddings to adequately distribute space. -The buttons in internship were running out of the box. I adjusted the internship boxes to accommodate all the content. -Adjusted the contact form & FAQ size and space -Other minimal changes to finish off.

Type of PR

Screenshots / videos (if applicable)

https://github.com/CSEdgeOfficial/learn.csedge.courses/assets/138357063/749aacce-e7db-4185-8850-88f0e344a9cc

Checklist:

github-actions[bot] commented 3 months ago

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

pankhuri92 commented 3 months ago

@CSEdgeOff, thank you for reviewing. Just to clarify the requested change - you mean to make it less congested? I'll work on the spacing then. Screenshot 2024-06-06 181307

Durgesh-Vaigandla commented 3 months ago

checkout this page

pankhuri92 commented 3 months ago

@CSEdgeOff I have made the required changes. I hope that suffices.

Screenshot 2024-06-06 210312

Screenshot 2024-06-06 210233

pankhuri92 commented 3 months ago

Hi @CSEdgeOff & @Durgesh-Vaigandla , I have made changes as requested and rest all seems to work fine. Could you please review my pr and let me know if any further improvement is needed?

CSEdgeOff commented 3 months ago

@pankhuri92 Update your PR so that We can be able to merge getting conflicts with your PR

pankhuri92 commented 3 months ago

@pankhuri92 Update your PR so that We can be able to merge getting conflicts with your PR

I have now resolved the conflicts. Please do let me know for further changes.

pankhuri92 commented 3 months ago

@CSEdgeOff, pls merge the pr if no further changes are required. Also could you assign level 3 to it...I had made it responsive in at least 4 versions which required a lot of detailed changes.

pankhuri92 commented 2 months ago

@CSEdgeOff, kindly merge my pr. It's been over a month.

github-actions[bot] commented 1 month ago

This issue has been automatically closed because it has been inactive for more than 30 days. If you believe this is still relevant, feel free to reopen it or create a new one. Thank you!