tudor/school_compare
1
0 Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request 'fix(modal): give the close button a 44px touch target' (#172) from fix/modal-close-tap-target into main
Stage (build -> staging -> E2E gate) / prepare (push) Successful in 1s
Details
Stage (build -> staging -> E2E gate) / Build Backend (FastAPI) (push) Successful in 19s
Details
Stage (build -> staging -> E2E gate) / Build Frontend (Next.js) (push) Successful in 1m25s
Details
Stage (build -> staging -> E2E gate) / Build Pipeline (Meltano + dbt + Airflow) (push) Successful in 13s
Details
Stage (build -> staging -> E2E gate) / Deploy to Staging (push) Successful in 27s
Details
Stage (build -> staging -> E2E gate) / E2E Journeys against Staging (push) Successful in 3m7s
Details

Browse Source 
Reviewed-on: #172
This commit was merged in pull request #172.
This commit is contained in:
tudor committed 2026-10-02 15:50:14 +00:00
2 parents 19c574edb0 27d83f9bd0
commit ccf0892a0e
1 file changed
+4
Download Patch File Download Diff File Expand all files Collapse all files
nextjs-app/components/Modal.module.css
+4
View File
@@ -79,7 +79,11 @@
font-family: var(--font-display);
}
/* 44px, the MOBILE.md touch-target minimum: the 24px icon and its padding
came to 40, and the phone filter sheet journey caught it. */
.closeButton {
min-width: 2.75rem;
min-height: 2.75rem;
padding: 0.5rem;
background: transparent;
border: none;