Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use a custom cluster marker style #1664

Merged
merged 2 commits into from
May 17, 2024
Merged

Use a custom cluster marker style #1664

merged 2 commits into from
May 17, 2024

Conversation

mxdvl
Copy link
Collaborator

@mxdvl mxdvl commented May 10, 2024

What does this change?

Before After
before after

How was this tested?

How have you tested your changes?

  • Locally with pnpm dev

Accessibility

If applicable to your changes, have you:

  • Tested with screen reader
  • Checked to see if navigable by keyboard
  • Ensured that the colour contrast is passing

Documentation

If applicable to your changes, have you:

  • Added/edited the appropriate documentation
  • Added any component(s) to Storybook

Copy link

vercel bot commented May 10, 2024

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

Name Status Preview Updated (UTC)
toiletmap ✅ Ready (Inspect) Visit Preview May 17, 2024 3:11pm

Copy link

📦 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 (🟡 +74 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!

Copy link

cypress bot commented May 10, 2024

Passing run #1181 ↗︎

0 63 0 0 Flakiness 0

Details:

Merge 04a5750 into bf1bff8...
Project: GBPTM Commit: 09a0aee7aa ℹ️
Status: Passed Duration: 01:58 💡
Started: May 17, 2024 3:14 PM Ended: May 17, 2024 3:16 PM

Review all test suite changes for PR #1664 ↗︎

Copy link

📦 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 (🟡 +74 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!

@mxdvl mxdvl changed the base branch from mxdvl/protomaps to main May 16, 2024 06:32
Copy link
Member

@ob6160 ob6160 left a comment

Choose a reason for hiding this comment

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

I really love this change! The style fits with the marker icons far better.
Happy to approve and merge if @rosejbon and @marxian are in agreement

Copy link
Collaborator

@marxian marxian left a comment

Choose a reason for hiding this comment

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

This looks great!

Thanks @mxdvl

@mxdvl mxdvl added the Chromatic starts the Chromatic GitHub action workflow label May 17, 2024
@mxdvl
Copy link
Collaborator Author

mxdvl commented May 17, 2024

I’ve accepted the Chromatic diffs as they were only changes to the OSM map updates – but there isn’t a story capturing the marker styles. Should I create one?

@ob6160
Copy link
Member

ob6160 commented May 17, 2024

I’ve accepted the Chromatic diffs as they were only changes to the OSM map updates – but there isn’t a story capturing the marker styles. Should I create one?

If it’s not too much extra effort to chuck in an extra map story with a marker cluster on it there’s no harm IMO.

I’m not massively bothered though as it should be quite obvious very quickly when testing if clusters look off if we capture it in Chromatic or not as it’s so core to the experience. Happy either way :)

@mxdvl
Copy link
Collaborator Author

mxdvl commented May 17, 2024

If it’s not too much extra effort to chuck in an extra map story with a marker cluster on it there’s no harm IMO.

I’ve tried a few things, but I’m not understanding how to add marker groups to a story, so I’d prefer getting this in first and possibly improving this as a follow up.

Copy link

📦 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 (🟡 +74 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!

@ob6160
Copy link
Member

ob6160 commented May 17, 2024

If it’s not too much extra effort to chuck in an extra map story with a marker cluster on it there’s no harm IMO.

I’ve tried a few things, but I’m not understanding how to add marker groups to a story, so I’d prefer getting this in first and possibly improving this as a follow up.

sounds good - let's merge it!

👏 first loo map contribution 🚽

we no longer differenciate
on the number of markers in a group
Copy link

📦 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 (🟡 +74 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!

@mxdvl mxdvl added Chromatic starts the Chromatic GitHub action workflow and removed Chromatic starts the Chromatic GitHub action workflow labels May 17, 2024
@mxdvl mxdvl merged commit a1b4e94 into main May 17, 2024
19 checks passed
@mxdvl mxdvl deleted the mxdvl/clusters branch May 17, 2024 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chromatic starts the Chromatic GitHub action workflow enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants