Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
09def26
map exploration, hide expired events, fix weback
chadbrokaw Aug 14, 2025
7060df9
Temporarily disable tests
chadbrokaw Aug 18, 2025
bb76f95
Revert webpack change
chadbrokaw Aug 18, 2025
5f0c362
Improve Search Header
chadbrokaw Aug 18, 2025
c64b311
fix: fix tests and cleanup code
chadbrokaw Sep 2, 2025
cb0661a
reenable tests
chadbrokaw Sep 2, 2025
3a40297
refactor: remove unused zoom change listener from SearchMap component
chadbrokaw Sep 2, 2025
9eb8bd6
fix: Fix BrowseResultsPage
chadbrokaw Sep 2, 2025
da2654a
fix: add some logging
chadbrokaw Sep 2, 2025
e4b1d20
fix: remove console log
chadbrokaw Sep 2, 2025
dc538be
attempt to fix category switching issue
chadbrokaw Sep 2, 2025
bac8449
Don't reset map init
chadbrokaw Sep 2, 2025
4d4e3c9
fix: remove logging
chadbrokaw Sep 2, 2025
a35790a
Pulling events from Open Data Portal
chadbrokaw Aug 29, 2025
baf1c5b
fix: add days of week and allow for series of events
chadbrokaw Sep 2, 2025
35929c7
fix: refresh styles
chadbrokaw Sep 15, 2025
1502628
fix: test
chadbrokaw Sep 15, 2025
38d53f3
fix: remove exploration work for eligibility filtering
chadbrokaw Sep 15, 2025
ef753be
fix: Fix some visual inconsistencies and bugs
chadbrokaw Sep 17, 2025
26c1577
rename featued events and shuffle homepage order
chadbrokaw Sep 18, 2025
b8d6546
fix: calendar section background color
chadbrokaw Sep 18, 2025
26613a6
Remove commented code
chadbrokaw Sep 18, 2025
0b20179
Merge branch 'main' into development
chadbrokaw Sep 18, 2025
d5f8563
Merge branch 'main' into development
chadbrokaw Sep 20, 2025
b3b9fae
fix: ensure https protocol
chadbrokaw Sep 20, 2025
836539b
fix: make it more robust and add security considerations
chadbrokaw Sep 20, 2025
b8132d4
Merge branch 'main' into development
chadbrokaw Sep 24, 2025
0358a39
fix: split apart the calendar
chadbrokaw Sep 24, 2025
d613616
Use React-GTM-Module
chadbrokaw Sep 24, 2025
dcd0c7a
fix: add error boundary and remove GA from development
chadbrokaw Sep 29, 2025
a84ee84
fix: Page not found
chadbrokaw Sep 29, 2025
553ad41
fix: Rearrange featured resources and the calendar
chadbrokaw Sep 29, 2025
a111a44
Review
chadbrokaw Sep 29, 2025
1f630ed
fix: comment
chadbrokaw Sep 29, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 3 additions & 18 deletions app/App.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import React, { useEffect, useState } from "react";
// Todo: Once GA sunsets the UA analytics tracking come July 2023, we can remove the "react-ga"
// package and all references to it:
// https://support.google.com/analytics/answer/12938611#zippy=%2Cin-this-article
import ReactGA_4 from "react-ga4";
import TagManager from "react-gtm-module";
import { Helmet } from "react-helmet-async";
import { useLocation } from "react-router-dom";
import { UserLocation, getLocation, websiteConfig, AppProvider } from "./utils";
Expand All @@ -18,6 +18,8 @@ import { AroundRadius } from "algoliasearch";
const { siteUrl, title } = websiteConfig;
export const OUTER_CONTAINER_ID = "outer-container";

TagManager.initialize({ gtmId: config.GOOGLE_ANALYTICS_GA4_ID });

export const App = () => {
const location = useLocation();
const [userLocation, setUserLocation] = useState<UserLocation | null>(null);
Expand All @@ -31,23 +33,6 @@ export const App = () => {
setUserLocation(userLocation);
setAroundLatLng(`${userLocation.coords.lat},${userLocation.coords.lng}`);
});

if (config.GOOGLE_ANALYTICS_GA4_ID) {
ReactGA_4.initialize(config.GOOGLE_ANALYTICS_GA4_ID);
}

setTimeout(() => {
// We call setTimeout here to give our views time to update the document
// title beforeGA sends its page view event
// TODO: This hack is old. Let's figure out if it is still necessary or
// there is a different modern approach
// (see: https://stackoverflow.com/questions/2497200/how-to-listen-for-changes-to-the-title-element/29540461#29540461)
const page = location.pathname + location.search;
ReactGA_4.send({
hitType: "pageview",
page,
});
}, 500);
}, [location, setAroundLatLng]);

if (!userLocation) {
Expand Down
17 changes: 11 additions & 6 deletions app/components/ui/Calendar/EventCalendar.module.scss
Original file line number Diff line number Diff line change
Expand Up @@ -401,9 +401,7 @@
justify-content: space-between;
padding: 1rem 0.5rem;
margin-bottom: 1rem;
background: linear-gradient(135deg, #f8f9fa 0%, #e9ecef 100%);
border-radius: 12px;
border: 1px solid #dee2e6;
border-bottom: 1px solid #dee2e6;
box-shadow: 0 2px 4px rgba(0, 0, 0, 0.05);

@media (min-width: 769px) {
Expand All @@ -415,8 +413,8 @@
background: white;
border: 2px solid #007bff;
border-radius: 50%;
width: 44px;
height: 44px;
width: 36px;
height: 36px;
display: flex;
align-items: center;
justify-content: center;
Expand Down Expand Up @@ -786,7 +784,8 @@
}

&:focus {
outline: none;
outline: 2px solid rgba(0, 123, 255, 0.3);
outline-offset: 4px;
border-radius: 50%;
}
&:active {
Expand Down Expand Up @@ -1052,3 +1051,9 @@
color: white;
}
}

.categoryInstructions {
margin-top: 8px;
margin-bottom: 24px;
Comment on lines +1056 to +1057
Copy link

Copilot AI Sep 29, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider using consistent spacing units throughout the stylesheet. The 8px and 24px values should ideally use a spacing scale (e.g., 0.5rem and 1.5rem) for better maintainability.

Suggested change
margin-top: 8px;
margin-bottom: 24px;
margin-top: 0.5rem;
margin-bottom: 1.5rem;

Copilot uses AI. Check for mistakes.
text-align: center;
}
Loading
Loading