Skip to content

Main Page Background changed#1389

Open
Ks103 wants to merge 4 commits into
SauravMukherjee44:mainfrom
Ks103:main
Open

Main Page Background changed#1389
Ks103 wants to merge 4 commits into
SauravMukherjee44:mainfrom
Ks103:main

Conversation

@Ks103
Copy link
Copy Markdown

@Ks103 Ks103 commented Jun 28, 2023

🛠️ 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)

  • My code follows the code style of this project.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

Screenshot (200)

@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 28, 2023

Deploy Preview for aec-library-wbsite ready!

Name Link
🔨 Latest commit 4174996
🔍 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
Copy link
Copy Markdown
Collaborator

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

@rohanbabbar04 rohanbabbar04 linked an issue Jun 28, 2023 that may be closed by this pull request
2 tasks
@Ks103
Copy link
Copy Markdown
Author

Ks103 commented Jun 28, 2023

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
Copy link
Copy Markdown
Author

Ks103 commented Jun 28, 2023

Commit*

@Ks103
Copy link
Copy Markdown
Author

Ks103 commented Jun 28, 2023

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

@rohanbabbar04 rohanbabbar04 changed the title updates Main Page Background changed Jun 29, 2023
@rohanbabbar04
Copy link
Copy Markdown
Collaborator

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

@Ks103
Copy link
Copy Markdown
Author

Ks103 commented Jun 30, 2023

I have aligned it in center

@rohanbabbar04
Copy link
Copy Markdown
Collaborator

rohanbabbar04 commented Jun 30, 2023

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

@Ks103
Copy link
Copy Markdown
Author

Ks103 commented Jun 30, 2023

done with the changes

@rohanbabbar04
Copy link
Copy Markdown
Collaborator

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

@rohanbabbar04
Copy link
Copy Markdown
Collaborator

Any updates on this @Ks103 ?

@Ks103
Copy link
Copy Markdown
Author

Ks103 commented Jul 4, 2023

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

@rohanbabbar04
Copy link
Copy Markdown
Collaborator

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..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug Report: Main page

2 participants