Travel_Website icon indicating copy to clipboard operation
Travel_Website copied to clipboard
verified publisher icon apu52

footer

Open kairveeehh opened this issue 1 year ago • 2 comments

Title and Issue number

#408 footer's improvement Title : Improved the footer

Issue No. : 408

Code Stack : html and css

Close #408

  1. fixed the problem of text line's changing on hovering the text
  2. added social icons for better UI
  3. improved the generalisation of colors
  4. added hover effects Screenshot 2024-05-15 143748 Screenshot 2024-05-15 143742 Screenshot 2024-05-15 143734

Checklist:

  • [ X] I have mentioned the issue number in my Pull Request.
  • [ XX] I have commented my code, particularly in hard-to-understand areas
  • [x] I have gone through the contributing.md file before contributing

Additional context (Mandatory )

Are you contributing under any Open-source programme?

gssoc

@apu52

please review and merge

kairveeehh avatar May 15 '24 09:05 kairveeehh

Deploy Preview for taupe-cendol-f7e2bb ready!

Name Link
Latest commit 88e08ded3fe33f5d7dfa21a92dcac4aba5a3ea63
Latest deploy log https://app.netlify.com/sites/taupe-cendol-f7e2bb/deploys/66447c9f753af10008d9ef31
Deploy Preview https://deploy-preview-447--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.

netlify[bot] avatar May 15 '24 09:05 netlify[bot]

@apu52

please review and merge it

also issues and conflicts arise when you merge the later ones first so please merge the ones which have been done first please

kairveeehh avatar May 15 '24 17:05 kairveeehh

@kairveeehh This PR has merge conflicts .please try to resolve the conflicts .So that I can merge & review the PR . Sorry for the late review . please resolve the conflicts

@apu52 sure I will fix it and tag you again

kairveeehh avatar May 22 '24 15:05 kairveeehh

@kairveeehh this PR still have merge conflicts . So , I just want to suggest you to make a new PR on this issue making a complete new branch in the forked repo. So that It will be helpful I am closing this PR for now. But you still have 5 more days to submit the PR

apu52 avatar May 22 '24 15:05 apu52

@kairveeehh this PR still have merge conflicts . So , I just want to suggest you to make a new PR on this issue making a complete new branch in the forked repo. So that It will be helpful I am closing this PR for now. But you still have 5 more days to submit the PR

Sure sure i will do it

kairveeehh avatar May 22 '24 15:05 kairveeehh