Skip to content

Complete UI Revamp: City Structure, Sticky Navbar, Search Bar & PHP Setup#28

Open
parul-64 wants to merge 1 commit intokomalkhatod1105:mainfrom
parul-64:solving-issue-#24
Open

Complete UI Revamp: City Structure, Sticky Navbar, Search Bar & PHP Setup#28
parul-64 wants to merge 1 commit intokomalkhatod1105:mainfrom
parul-64:solving-issue-#24

Conversation

@parul-64
Copy link

@parul-64 parul-64 commented Aug 1, 2025

This pull request introduces a major enhancement to the "One Stop Guide" project by improving the site structure, design, and interactivity for better scalability and user experience.

What's Done

Created modular folder structure for each city: /cities/city-name/index.html, style.css, script.js
Converted all city pages into PHP templates (e.g., sikar.php) to support future dynamic data
Implemented a sticky navigation bar with clean layout and improved link spacing
Redesigned footer with contact info, quick links, and social media integration
Added a search bar to quickly find cities by name

###Files Added/Modified

/cities/**/index.html
/partials/header.php, /partials/footer.php
index.html, style.css, script.js
Individual city .php files

Future Enhancements

City info expansion: culture, local tips, embedded maps
Visual media: sliders, videos
Plan Trip checklist, feedback form

Please review and suggest changes if needed.

@komalkhatod1105
Copy link
Owner

@parul-64 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.

@parul-64
Copy link
Author

parul-64 commented Aug 4, 2025

I haven't made any changes to index.html after opening my PR.

@komalkhatod1105
Copy link
Owner

@parul-64 ok

@parul-64
Copy link
Author

parul-64 commented Aug 4, 2025

okay ,just a small thing I noticed the issue doesn't have any label yet would it be possible to add.

@parul-64
Copy link
Author

parul-64 commented Aug 8, 2025

hello @komalkhatod1105 just checking in regarding this PR. I've been waiting for any updates kindly let me know if there is anything else I need to do from my side .Also it would be great if you could review and merge it whenever you get chance .

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