Skip to content

Port menu pinning and map bugfixes from Forum2025 - #44

Open
eduardoperetto wants to merge 35 commits into
mainfrom
feat/menu-pinning-from-forum
Open

Port menu pinning and map bugfixes from Forum2025#44
eduardoperetto wants to merge 35 commits into
mainfrom
feat/menu-pinning-from-forum

Conversation

@eduardoperetto

@eduardoperetto eduardoperetto commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Adds menu pinning (lock/unlock) with a separate Pinned Menus sidebar while preserving the existing tag detail sidebar.
  • Ports map/menu bugfixes from Forum2025: KML when another menu is pinned, simultaneous links only when both endpoints are on map, pinned-menu marker redraw, null-safe link removal, first-menu-per-group init, dashed-link styling, and smaller pin markers.
  • Adds KML serializer URL method for correct media serialization.

Excluded (forum-only): Ipê/CNDs special menu rules, nginx/backup env edits, attributionControl:false, and Forum’s tag-sidebar replacement.

Test plan

  • Load map and confirm first menu of first group is selected
  • Pin/unpin menus; verify pinned markers stay visible when switching menus
  • Unpin from sidebar and confirm map refreshes immediately
  • Open tag sidebar by clicking a tag; confirm description/locations still show
  • With links enabled, verify links only draw when both locations are on map
  • Pin a menu with KML shapes; switch to another menu and confirm KML still renders
  • Toggle close/open all eyes per menu; confirm link groups follow tag/KML visibility

leonardolauryel and others added 28 commits April 8, 2025 18:12
* Fix: Adding validation to domain names

* Changing string to English

* fix: sync owner changes and prevent inconsistencies in multitenancy

- Sync many-to-many relationship (clients_tenantuser_tenants) when client owner changes
- Sync permissions in tenant schema (permissions_usertenantpermissions) when owner changes
- Prevent removing tenants from users who own them with form validation

* fix: prevent error when creating new tenant users"

* fix: sync tenant permissions on user creation and removal

* refactor: apply clean code principles to multitenancy admin
@eduardoperetto

This comment was marked as resolved.

@eduardoperetto

This comment was marked as resolved.

eduardoperetto and others added 7 commits July 24, 2026 17:13
Extract the general-purpose pinning feature, KML/link/marker fixes, and per-menu visibility behavior onto main while omitting forum-specific hardcoding.
Allow long menu names to wrap, nudge the collapser, and adjust left-menu padding.
@eduardoperetto
eduardoperetto force-pushed the feat/menu-pinning-from-forum branch from d1cf398 to e3a3abc Compare July 24, 2026 20:20
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.

3 participants