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
Feature/add change area overlay - ticket#135 #78
Conversation
…ts to change area on map on categoryId/locations/locationId route
✔️ Deploy Preview for ons-data-map ready! 🔨 Explore the source changes: e9fa528 🔍 Inspect the deploy log: https://app.netlify.com/sites/ons-data-map/deploys/61e010e797409f0008d4a312 😎 Browse the preview: https://deploy-preview-78--ons-data-map.netlify.app |
Blocked by ONSdigital/design-system#1761 (comment) |
Fix incoming in ONSdigital/design-system#1798 |
…routes/area/index.svelte pages to toggle the header when the user wants to update the location
…s/area/index.svelte page
…s/[topicSlug]/[tableSlug]/[categorySlug].svelte page
…aHeader component
… routes/[topicSlug]/[tableSlug]/[categorySlug].svelte to display the change
…reaComponent error panel
…mponent error panel
…/[categorySlug].svelte screen
Now unblocked. |
…pecific to the selected geography
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.
👍 LGTM
What
I have added the logic to toggle the header on Explore by Location (Walsall) and Explore by Category & Location screens when a user wants to search for a different location.
I haven't added the styling, I leave this with you @designorant because when I am trying to use a position on the header everything moves around the page...
Note: The js does not load on the
<ONSAutosuggest/>
component on anon:click
event because at the moment we can only load the ons DS js at a page level not at a component level and we hope that this will go away once the issue is fixed by the design system team.