Skip to content

Add more cities, enhance the UI , Add more information and photos for each city #22#31

Open
gayatrijangid wants to merge 2 commits intokomalkhatod1105:mainfrom
gayatrijangid:main
Open

Add more cities, enhance the UI , Add more information and photos for each city #22#31
gayatrijangid wants to merge 2 commits intokomalkhatod1105:mainfrom
gayatrijangid:main

Conversation

@gayatrijangid
Copy link

@gayatrijangid gayatrijangid commented Aug 1, 2025

It is issue #22
Hi [Komal Khatod] @komalkhatod1105

I’ve completed the assigned enhancement task. Here’s a summary of what I’ve implemented:

✅ What I Did
As cities are given in the project, including:
Ajmer, Kishangarh ,Laxmangarh , Kota ,Delhi ,Fatehpur, Ratangarh

Enhanced city descriptions with detailed and structured information (important facts, medium-level info, and popular places).

Improved UI layout for the gallery section using responsive CSS (grid layout, image hover effect, consistent design).

Added high-quality images (5–6 for each city) to visually enrich the content.

Maintained accessibility and semantic HTML.

**** I Had Change layout of cities because to match UI of home page and city pages *****

🔍 Things to Review
Visual layout and responsiveness
Image alt texts and loading behavior
City content accuracy

Let me know if anything needs to be adjusted or improved. Looking forward to your feedback!

Thanks,
[Gayatri Jangid]

@gayatrijangid gayatrijangid changed the title Enhancement: Cities, UI Improvements, and Gallery Enhancement: Cities, UI Improvements, and Gallery #22 Aug 1, 2025
@gayatrijangid gayatrijangid changed the title Enhancement: Cities, UI Improvements, and Gallery #22 Add more cities, enhance the UI , Add more information and photos for each city #22 Aug 1, 2025
@komalkhatod1105
Copy link
Owner

@gayatrijangid Hey! I'm getting a merge conflict in indx.html. Looks like there are conflicting changes from both branches. I'm resolving it right now — let me know if you’ve made any important edits so I don’t overwrite them.

@gayatrijangid
Copy link
Author

gayatrijangid commented Aug 4, 2025

No I had not made changes in that only I had change file name from indx to index.html
Because this file name is simpler than that
@komalkhatod1105

@gayatrijangid
Copy link
Author

@komalkhatod1105 you can change file name plz...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants