newmap #2
Merged
newmap #2
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull Request Overview
This pull request introduces significant enhancements to the MAC Paving and Sealcoating website by improving map functionality, updating component integration, adjusting SEO settings, and overhauling the project documentation.
- Added a reusable Map component (with support for multiple containers and controlled z-index).
- Replaced iframe-based maps with the new component implementation and applied specific CSS fixes.
- Updated meta tags, robots exclusion rules, and documentation to reflect the new branding and enhanced functionality.
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/pages/contact.astro | Adjusted asset path using an alias and reformatted map background markup for consistency. |
| src/components/MapSection.astro | Integrated the new Map component in place of an iframe and added inline z-index styles. |
| src/components/Map.astro | Introduced a new reusable Map component with Leaflet integration and associated CSS/JS inclusions. |
| src/components/Footer.astro | Replaced the iframe with the Map component and updated the designer hyperlink. |
| public/robots.txt | Updated URL blocking rules to include /map.html. |
| public/map.html | Added a robots meta tag and removed outdated resource links to improve SEO. |
| public/assets/js/map.js | Refactored map initialization for multiple containers and enhanced popup formatting. |
| public/assets/css/map.css | Added z-index styles to ensure proper layering of map elements. |
| public/assets/css/map-fix.css | Included comprehensive z-index fixes for Leaflet subcomponents. |
| README.md | Overhauled documentation with new project branding, feature descriptions, and updated tech stack. |
Comments suppressed due to low confidence (2)
src/pages/contact.astro:15
- Ensure that the alias '@' is correctly configured in the project configuration for reliable asset resolution.
import contactBg from "@/assets/imgs/bg/contact-cover.webp";
src/components/Footer.astro:145
- The change of the designer hyperlink from a URL to a telephone link may be unintended; please verify if this update was intentional.
href="tel:+522201758149"
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces significant enhancements to the MAC Paving and Sealcoating website, focusing on improving the map functionality, restructuring components, optimizing SEO, and refining the project's documentation. Key changes include the addition of a reusable
Mapcomponent, fixes for z-index issues in map styling, and updates to theREADME.mdfile to reflect the project's new scope and features.Map Functionality Enhancements:
src/components/Map.astro: Added a reusableMapcomponent for embedding maps with controlled z-index and responsive styling. This replaces the iframe-based implementation and ensures consistent behavior across pages.public/assets/css/map-fix.css: Introduced specific styles to resolve z-index conflicts between the map and other elements, ensuring proper layering and visibility of tooltips and popups.public/assets/js/map.js: Refactored the map initialization logic to support multiple map containers and improved popup content formatting for better user experience. [1] [2] [3] [4]SEO and Accessibility Improvements:
public/map.html: Removed unnecessary meta tags and addedrobotsdirectives to prevent indexing of the map page, aligning with SEO best practices. [1] [2]public/robots.txt: Blocked outdated URLs, includingmap.html, ensuring clean and optimized search engine visibility.Component Restructuring:
src/components/Footer.astro: Replaced the iframe-based map in the footer with the newMapcomponent for better integration and styling consistency.src/components/MapSection.astro: Integrated theMapcomponent into the service areas section, streamlining map embedding across the site.Documentation Updates:
README.md: Overhauled the file to reflect the project's new branding, features, and tech stack. Added detailed sections on setup, responsive design, performance, and deployment.