Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug fix/map #9

Closed
wants to merge 17 commits into from
Closed

Bug fix/map #9

wants to merge 17 commits into from

Conversation

AdityaSingh-7
Copy link
Contributor

Added India-Map Component

Description

This pull request adds improvements to the India map project in Astro, including updates to the CSS for better responsiveness, enhancements to the SVG map layout, and integration of JSON data for displaying state-specific folk dances. The changes aim to provide a smoother user experience and better visuals across devices.

Fixes #14 (Improve SVG and image layout responsiveness)

Current Issue

Facing problem in dynamic display of the pop-up on the map page.

Example : The pop up is only visible in the upper part of the map. Hence, one needs to scroll up to view the pop up.

How Has This Been Tested?

The changes have been tested using the following methods:

  • Verified responsive design across different screen sizes (mobile, tablet, desktop)
  • Ensured SVG map renders correctly and is interactive
  • Tested integration of JSON data for state-specific dance information
  • Checked compatibility on Chrome, Firefox, and Safari
  • Performed accessibility checks using Lighthouse

Test Configuration:

  • Browser(s) tested: Chrome, Firefox, Safari
  • Device(s) tested: Desktop (macOS, Windows), Mobile (iOS, Android)

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

Additional Notes

These changes ensure the map project aligns with the goals of a visually appealing, responsive, and content-rich web experience for showcasing Indian folk dances.

@recursivezero
Copy link
Owner

Resolve conflicts

Copy link
Collaborator

@xkeshav xkeshav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix all ASAP

---
import BaseLayout from "@/layouts/Base"
import stateData from "@/assets/map_assets/stateData.json";
import "@/assets/styles/map_styles/stateStyles.css";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no nested css files

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

resolved folder structure

<TitleLayout title="Explore Dances of India" />
<IndiaMap />
<Popup />
<!-- <Icon name="Andhra" /> -->
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove commented code

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed commented code

@xkeshav xkeshav closed this Jan 29, 2025
@xkeshav xkeshav deleted the bugFix/map branch January 29, 2025 18:14
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.

3 participants