public-convenience-ltd / toiletmap

API/UI server for the Great British Public Toilet Map
https://www.toiletmap.org.uk
MIT License
42 stars 9 forks source link

Use light map styles #1668

Closed mxdvl closed 3 months ago

mxdvl commented 4 months ago

What does this change?

A mix of greys, greens and blues, which offers better contrast with the surrounding interface.

image

How was this tested?

How have you tested your changes? Ways of testing could include:

Accessibility

If applicable to your changes, have you:

Documentation

If applicable to your changes, have you:

vercel[bot] commented 4 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
toiletmap ✅ Ready (Inspect) Visit Preview May 14, 2024 6:59am
github-actions[bot] commented 4 months ago

📦 Next.js Bundle Analysis for toiletmap

This analysis was generated by the Next.js Bundle Analysis action. 🤖

⚠️ Global Bundle Size Increased

Page Size (compressed)
global 194.68 KB (🟡 +1 B)
Details

The global bundle is the javascript bundle that loads alongside every page. It is in its own category because its impact is much higher - an increase to its size means that every page on your website loads slower, and a decrease means every page loads faster.

Any third party scripts you have added directly to your app using the <script> tag are not accounted for in this analysis

If you want further insight into what is behind the changes, give @next/bundle-analyzer a try!

cypress[bot] commented 4 months ago

4 failed tests on run #1167 ↗︎

4 44 0 15 Flakiness 0

Details:

Merge bb3ec3f2d3212f00d9e32777f0b6de3cb0fcd2f2 into 249e8182886615ca05c441e1a818...
Project: GBPTM Commit: 228c0a14b8 ℹ️
Status: Failed Duration: 02:04 💡
Started: May 14, 2024 7:05 AM Ended: May 14, 2024 7:07 AM
Failed  desktop/add.cy.ts • 1 failed test • 1. Chrome View Output
Test Artifacts
Adding new toilets to the platform > Desktop > should load the add toilet page successfully for authenticated users
Failed  mobile/edit.cy.ts • 1 failed test • 1. Chrome View Output
Test Artifacts
Edit page tests > Mobile > should load the edit page with expected data given a valid toilet id
Failed  mobile/add.cy.ts • 1 failed test • 1. Chrome View Output
Test Artifacts
Adding new toilets to the platform > Mobile > should load the add toilet page successfully for authenticated users
Failed  mobile/remove.cy.ts • 1 failed test • 1. Chrome View Output
Test Artifacts
Remove page tests > Mobile > should remove a toilet successfully when going through the removal flow

Review all test suite changes for PR #1668 ↗︎

mxdvl commented 3 months ago

Superceded by #1670