-
Notifications
You must be signed in to change notification settings - Fork 50
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
Bump mazemap from 2.0.63 to 2.0.96 #4170
Conversation
5f6816a
to
5280a2f
Compare
@@ -87,6 +87,7 @@ function mazemapAutocomplete<Props>({ | |||
), | |||
}); | |||
}) | |||
.catch((error) => {}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the point of this? You don't do anything with the error?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, but something seems to have changed about the search in mazemap so it will throw something like "search cancelled because a new one was initiated" when writing (because that triggers new searches)
@@ -176,6 +184,7 @@ export const MazemapEmbed = ({ mazemapPoi, ...props }: Props) => { | |||
touchAction: 'pan-x pan-y', | |||
}} | |||
id="mazemap-embed" | |||
className={styles.mazemapEmbed} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand what this styling is affecting? The span below? Because that has already a color: #111 by the blockingText
class?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It makes the color of the other text in the embed also black, even on dark mode. Previously zLevelControl (floor selector) got white text on white bg on dark mode.
1120eea
to
4313794
Compare
939f88f
to
58f0c08
Compare
Description
Bumps mazemap from 2.0.63 to 2.0.96
Release notes
Sourced from mazemaps api docs.
Result
Seemingly everything works like before but it should now use a newer mapbox version.
(Newest version doesnt work with our webpack config)
Testing
Map and search still works