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

@Ks103 Ks103 commented Jun 28, 2023

Copy link
Copy Markdown

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

netlify Bot commented Jun 28, 2023

Copy link
Copy Markdown

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

Ks103 commented Jun 28, 2023

Copy link
Copy Markdown
Author

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

Ks103 commented Jun 28, 2023

Copy link
Copy Markdown
Author

Commit*

@Ks103

Ks103 commented Jun 28, 2023

Copy link
Copy Markdown
Author

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

Ks103 commented Jun 30, 2023

Copy link
Copy Markdown
Author

I have aligned it in center

@rohanbabbar04

rohanbabbar04 commented Jun 30, 2023

Copy link
Copy Markdown
Collaborator

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

@Ks103

Ks103 commented Jun 30, 2023

Copy link
Copy Markdown
Author

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

Ks103 commented Jul 4, 2023

Copy link
Copy Markdown
Author

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