Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Sep 21, 2025

Removed the "Group Locations" section containing the interactive Leaflet map from the bottom of the people page as requested. This section included map visualization of the University of Texas at Austin location with various tile provider fallbacks.

Changes Made

  • Removed the entire "Group Locations" section from people/index.qmd (lines 95-182)
  • Eliminated Leaflet CSS and JavaScript library imports
  • Removed map container HTML div and associated styling
  • Cleaned up JavaScript code for map initialization, tile providers, markers, and error handling

Before/After

Before: The people page ended with an interactive map showing the group's location in Austin, TX

After: The people page now ends cleanly with the Alumni section, providing a cleaner and more focused layout

The page continues to display all personnel sections (Principal Investigator, Research Scientists, Postdoctoral Fellows, Graduate Students, Post Baccalaureate Researchers, and Alumni) without any functional changes to those sections.

People page after map removal

Testing

  • ✅ Site builds successfully with quarto render
  • ✅ All existing functionality preserved
  • ✅ No map-related content remains in generated HTML
  • ✅ Page layout remains clean and professional

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@Copilot Copilot AI changed the title [WIP] Remove the map from the bottom of the people page Remove interactive map from people page Sep 21, 2025
@Copilot Copilot AI requested a review from JacobKMcPherson September 21, 2025 20:25
Copilot finished work on behalf of JacobKMcPherson September 21, 2025 20:25
@JacobKMcPherson JacobKMcPherson marked this pull request as ready for review September 22, 2025 15:31
@Copilot Copilot AI review requested due to automatic review settings September 22, 2025 15:31
@JacobKMcPherson JacobKMcPherson merged commit 04edae3 into main Sep 22, 2025
5 checks passed
@JacobKMcPherson JacobKMcPherson deleted the copilot/fix-c93bf863-f6ea-4617-8b45-62bfaa8bc4aa branch September 22, 2025 15:31
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the interactive map functionality from the people page by eliminating the "Group Locations" section. The change simplifies the page layout by removing the Leaflet.js map visualization that previously showed the University of Texas at Austin location.

  • Removed the entire "Group Locations" section containing the interactive map
  • Eliminated all Leaflet CSS and JavaScript dependencies
  • Cleaned up map-related HTML and JavaScript code

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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