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
186 stars 407 forks source link

feat: Trip Gallery Page is Been Updated #946

Open AbheetHacker4278 opened 3 days ago

AbheetHacker4278 commented 3 days ago

Title and Issue number

Title :

[Feature Request]: Trip Gallery Changes

Issue No. :

925

Code Stack :

Close #<#925>

Description

Here in this PR i have do some of the major changes in Trip Gallery , i have improved the User Interface of the Page , by using cool animation , so due to this the experience of the users also get enhanced.

Video/Screenshots (mandatory)

BEFORE Screenshot 2024-06-27 230043

AFTER image

Video

https://github.com/apu52/Travel_Website/assets/114290748/e4dbeb5c-48c8-48f0-a25f-a12810c9a943

Type of PR

Checklist:

Additional context:

@apu52 please review this PR and merge with main branch under Gssoc24

Are you contributing under any Open-source programme?

GSSOC24

netlify[bot] commented 3 days ago

Deploy Preview for taupe-cendol-f7e2bb ready!

Name Link
Latest commit 06687b9c47e073c1ea8a6acaa351cb3ebd6f37e5
Latest deploy log https://app.netlify.com/sites/taupe-cendol-f7e2bb/deploys/6682a3c06fbf0300081a86db
Deploy Preview https://deploy-preview-946--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.

itsekta commented 2 days ago

Hey @AbheetHacker4278, can you add an overlay layer on these card images so that the content on the images will be move visible on hovering on it

AbheetHacker4278 commented 2 days ago

Hey @AbheetHacker4278, can you add an overlay layer on these card images so that the content on the images will be move visible on hovering on it

Okay i will do it

AbheetHacker4278 commented 1 day ago

@itsekta I have done the necessary changes , please review it and merge it with main branch

AbheetHacker4278 commented 1 day ago

@itsekta please also add Gssoc labels with level 2

AbheetHacker4278 commented 1 day ago

@itsekta if you like the changes please merge with main branch

AbheetHacker4278 commented 16 hours ago

@itsekta @apu52 please merge into main branch if there is no errors

itsekta commented 11 hours ago

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

AbheetHacker4278 commented 11 hours ago

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

Okay i will do that in new PR , Because this take some time and i also have to re structure the code base from start to end , so better you merge this pr with main branch and tommorow i will open an new issue you just approve it, is it fine.

AbheetHacker4278 commented 11 hours ago

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

Because code base are in very bad structure very messy , thats why i am asking you

AbheetHacker4278 commented 11 hours ago

Hey, @AbheetHacker4278 can you also make it responsive for mobile screen as well

Please merge this PR , tommorow i will open new issue and accept it also πŸ™