apu52 / Travel_Website

πŸš€ Welcome🌈 , This repo is officially accepted for IEEE-IGDTUW Open Source Week 2024🌟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
302 stars 591 forks source link

fix: fixed-directing-to-different-pages-from-plan-a-trip-page-and-image-slider-to-stop-when-it-reaches-the-last-image #1588

Closed Arveti-likhitha closed 3 weeks ago

Arveti-likhitha commented 3 weeks ago

Title : Enhancing Plan a trip page

Issue No. : #1493

Code Stack : HTML, CSS, JS

Close #1493

Description

Made changes to make sure they are navigating to the correct pages and changing the image slider such that navigation stops after it reaches the last image card.

Video/Screenshots (mandatory)

https://github.com/user-attachments/assets/fc8397df-7378-4064-aae0-c859e4efc490

Type of PR

Checklist:

Additional context:

Are you contributing under any Open-source programme?

No

github-actions[bot] commented 3 weeks ago

Thank you for submitting your pull request! πŸ™Œ We'll review it as soon as possible. In the meantime, If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

netlify[bot] commented 3 weeks ago

Deploy Preview for taupe-cendol-f7e2bb ready!

Name Link
Latest commit 4059143554d4832d9d6e71d9289f8ee2da75df9a
Latest deploy log https://app.netlify.com/sites/taupe-cendol-f7e2bb/deploys/6722008c76ba3600080cd44c
Deploy Preview https://deploy-preview-1588--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.

Arveti-likhitha commented 3 weeks ago

@apu52 I have made changes. Can you check and let me know if there are any conflicts?

apu52 commented 3 weeks ago

@Arveti-likhitha the conflicts are still present in this repo , please try to resolve them asap

Arveti-likhitha commented 3 weeks ago

@apu52 I resolved all the conflicts except for one. There is a conflict with the part of the code which is not present in the "index.html" file. So I am not able to resolve the conflict.

Could you guide me on how to resolve this?

apu52 commented 3 weeks ago

@Arveti-likhitha u can do one thing , try to close this PR for now , Try to make a new branch on your forked repo & then make all of your proposed changes in that branch & try to push all of your commits in that branch & make/submit a new pr.