apu52 / Travel_Website

๐Ÿš€ Welcome๐ŸŒˆ , This repo is officially accepted for GSSOC'2k24๐ŸŒŸ & VsOC'24๐ŸŒŸThis repo contains my project on Travel Website โœˆ๏ธ using HTML,CSS . It contains the landing page of a travel tour company website. If you have any front end project on this or you can add new features in the project or in this repos.
https://apu52.github.io/Travel_Website/
MIT License
196 stars 422 forks source link

fix: Footer Section alignment issue fixed #982

Closed TBorundia closed 2 days ago

TBorundia commented 4 days ago

Title and Issue number

Title : Footer Section alignment Issue No. : #906

Code Stack :

Close #906

Description

1.Alignment issue of the text and links.

  1. Icons where displaying twice.
  2. Google translator script working .
  3. Hovering effect to icons according to there logo.
  4. Made the page responsive too.

Video/Screenshots (mandatory)

Before: Screenshot 2024-07-04 013705

After:

https://github.com/apu52/Travel_Website/assets/168854094/97798ac0-2679-4b09-817f-c3143a7dce50

Also made is responsive for small screen: Screenshot 2024-07-04 124440

Type of PR

Checklist:

Are you contributing under any Open-source programme?

netlify[bot] commented 4 days ago

Deploy Preview for taupe-cendol-f7e2bb ready!

Name Link
Latest commit 1d11d64900f60edd49dc99ea34294a00200f576b
Latest deploy log https://app.netlify.com/sites/taupe-cendol-f7e2bb/deploys/6686ed7d3eb7900008a86777
Deploy Preview https://deploy-preview-982--taupe-cendol-f7e2bb.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.

TBorundia commented 4 days ago

@apu52 @itsekta , please review this PR. Thank You!

itsekta commented 4 days ago

Hey @TBorundia, can you also look for this additional space in footer is doesn't looks nice image

TBorundia commented 4 days ago

@itsekta , which space mam? The space , beside of 3 lines of header section?

itsekta commented 4 days ago

The white space

TBorundia commented 4 days ago

The white space

Ok mam, I will also solve this issue.

apu52 commented 4 days ago

The white space

Ok mam, I will also solve this issue.

have u done the changes which @itsekta ma'am have already mentioned?? If yes then just reply So that I can review your PR

TBorundia commented 2 days ago

The white space

Ok mam, I will also solve this issue.

have u done the changes which @itsekta ma'am have already mentioned?? If yes then just reply So that I can review your PR

@apu52 @itsekta , I have made the changes and removed the white space also that mam asked for. Please review this PR.