Skip to content

Add openstreetmap to locations and Wrap blog tag filters in nav element for accessibility#639

Closed
Kkt04 wants to merge 2 commits intowagtail:mainfrom
Kkt04:add-openstreetmap-to-locations
Closed

Add openstreetmap to locations and Wrap blog tag filters in nav element for accessibility#639
Kkt04 wants to merge 2 commits intowagtail:mainfrom
Kkt04:add-openstreetmap-to-locations

Conversation

@Kkt04
Copy link

@Kkt04 Kkt04 commented Mar 8, 2026

Fixes #625

Issue: Add OpenStreetMap embed to Location pages for geographic context (zero JavaScript, no API key required) and Wrap blog tag filters in nav element for accessibility

Changes Made:

  1. bakerydemo/locations/models.py
    • Removed Google Maps API key reference from context
    • Added lat and long properties for cleaner template access
  2. bakerydemo/templates/locations/location_page.html
    • Added OpenStreetMap iframe showing the specific location
    • Added "View on OpenStreetMap" external link
  3. bakerydemo/templates/locations/locations_index_page.html
    • Added OpenStreetMap iframe showing first location on index page
  4. bakerydemo/static/css/main.css
    • Added styles for .location-map, .location-map__link, .location-index-map, .location-index-map__note

Why this approach:

  • Zero JavaScript (matches project policy)
  • No API key required (unlike Google Maps)
  • Fully accessible, degrades gracefully if iframe is blocked

Also I edited the file bakerydemo/templates/blog/blog_index_page.html to wrap the blog tag filters in a

element for accessibility
Git Branch:

  • Branch name: add-openstreetmap-to-locations

@Kkt04 Kkt04 changed the title Add openstreetmap to locations Add openstreetmap to locations and Wrap blog tag filters in nav element for accessibility Mar 8, 2026
@Kkt04
Copy link
Author

Kkt04 commented Mar 10, 2026

@thibaudcolas kindly review the PR and let me know if i need to make any changes

@thibaudcolas
Copy link
Member

@Kkt04 👋 I’m not sure why you’re working on this? This was just a suggestion someone made and it’s not particularly well suited to this project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Google Maps section to display bakery locations

2 participants