search

SauravMukherjee44 / Aec-Library-Website

This is an Open-Source Library Website in which you get Resources to learn different topics, Donate book section to donate your old books, and a Book issue section to keep a record of all the books issued. -HacktoberFest Accepted
https://aec-library-wbsite.netlify.app/
MIT License
106 stars 242 forks source link

Main Page Background changed #1389

Open Ks103 opened 1 year ago

Ks103 commented 1 year ago

🛠️ Fixes Issue #1377

The main page background changed and the alignments of the boxes in main page are now properly aligned

👨‍💻 Changes proposed

The main page is changed as per the requirement

✔️ Check List (Check all the applicable boxes)

📄 Note to reviewers

📷 Screenshots

Screenshot (200)

netlify[bot] commented 1 year ago

Deploy Preview for aec-library-wbsite ready!

Name Link
Latest commit 4174996d26025fe15eee78b51bfe5495f9d25fcf
Latest deploy log https://app.netlify.com/sites/aec-library-wbsite/deploys/64a05cda5ca76b000835bdee
Deploy Preview https://deploy-preview-1389--aec-library-wbsite.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

rohanbabbar04 commented 1 year ago

Can you please write a more appropriate title Updates does'nt tell anything, a better name would be good....

Ks103 commented 1 year ago

Actually sir I have comment it as updates so Now I can't change the comment

It specifies that I have updated the css files

Ks103 commented 1 year ago

Commit*

Ks103 commented 1 year ago

I request you to please check the files and merge my issue Thankyou.

rohanbabbar04 commented 1 year ago

image Reduce margin from top as you can see the table is not aligned properly...

Ks103 commented 1 year ago

I have aligned it in center

rohanbabbar04 commented 1 year ago

@Ks103 you have merge conflicts do resolve.... Also reduce blur of the background image..

Ks103 commented 1 year ago

done with the changes

rohanbabbar04 commented 1 year ago

align the below box centerally to the outer box, also check for different screen size image

rohanbabbar04 commented 1 year ago

Any updates on this @Ks103 ?

Ks103 commented 1 year ago

It is done @rohanbabbar04 Request you to please merge the issue

rohanbabbar04 commented 1 year ago

align the below box centerally to the outer box, also check for different screen size image

I don't see this change @Ks103, the box does not look centrally aligned and for smaller screen sizes it is not apt..