Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Main Page Background changed #1389

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

Ks103
Copy link

@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

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

Ks103 commented Jun 28, 2023

Commit*

@Ks103
Copy link
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
Collaborator

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

@Ks103
Copy link
Author

Ks103 commented Jun 30, 2023

I have aligned it in center

@rohanbabbar04
Copy link
Collaborator

rohanbabbar04 commented Jun 30, 2023

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

@Ks103
Copy link
Author

Ks103 commented Jun 30, 2023

done with the changes

@rohanbabbar04
Copy link
Collaborator

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

@rohanbabbar04
Copy link
Collaborator

Any updates on this @Ks103 ?

@Ks103
Copy link
Author

Ks103 commented Jul 4, 2023

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

@rohanbabbar04
Copy link
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