-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
✔️ Deploy Preview for ons-data-map ready! 🔨 Explore the source changes: 7b09d83 🔍 Inspect the deploy log: https://app.netlify.com/sites/ons-data-map/deploys/619b0f1b715fcf00082476fc 😎 Browse the preview: https://deploy-preview-67--ons-data-map.netlify.app |
Signed-off-by: Tom Ridd <twridd@gmail.com>
Signed-off-by: Tom Ridd <twridd@gmail.com>
Signed-off-by: Tom Ridd <twridd@gmail.com>
Signed-off-by: Tom Ridd <twridd@gmail.com>
Signed-off-by: Tom Ridd <twridd@gmail.com>
Signed-off-by: Tom Ridd <twridd@gmail.com>
Signed-off-by: Tom Ridd <twridd@gmail.com>
Signed-off-by: Tom Ridd <twridd@gmail.com>
Signed-off-by: Tom Ridd <twridd@gmail.com>
Signed-off-by: Tom Ridd <twridd@gmail.com>
b1450e0
to
11d6592
Compare
Signed-off-by: Tom Ridd <twridd@gmail.com>
Signed-off-by: Tom Ridd <twridd@gmail.com>
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.
This is totally out of my breath, really but it looks great. A very welcome refactor / legacy distinction, docs (!!!), code comments, config in place, and tests!
Splendid stuff!
code={`<script> | ||
let radioValue = "none"; | ||
</script> | ||
code={`<script ✂prettier:content✂="CiAgICAgIGxldCByYWRpb1ZhbHVlID0gIm5vbmUiOwogICA=">{}</script> |
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.
This is a bug in Prettier's Svelte Plugin. We've removed that <script/>
tag from there as a temporary workaround.
# Map | ||
|
||
The map has been refactored to make it easier to use. There is no html in the MapComponent.svelte components which makes | ||
this relaltively easy |
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.
l 😅
Wouldn't bother pointing this out but wanted a space to say this doc is great!
This PR is massive but puts in place a data model and map functionality.
I'm sure there are features that are missing but if we can pay some quick attention and use our components we will be able to put out full user journeys this sprint