Skip to content

Fix geoman draw control collapsed bug#111

Merged
giswqs merged 3 commits intomainfrom
collapse
Nov 8, 2025
Merged

Fix geoman draw control collapsed bug#111
giswqs merged 3 commits intomainfrom
collapse

Conversation

@giswqs
Copy link
Copy Markdown
Member

@giswqs giswqs commented Nov 8, 2025

No description provided.

Copilot AI review requested due to automatic review settings November 8, 2025 00:31
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors the add_geoman_control method by removing the inline configuration of collapse-related settings and replacing it with a call to collapse_geoman_control() method after the control is added.

Key changes:

  • Removed inline configuration of controlsCollapsible and controlsUiEnabledByDefault settings
  • Removed collapsed field from control_options payload
  • Added call to collapse_geoman_control() method when collapsed is truthy

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread anymap/maplibre.py Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 8, 2025

@github-actions github-actions Bot temporarily deployed to pull request November 8, 2025 00:38 Inactive
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@giswqs giswqs merged commit d49117f into main Nov 8, 2025
6 of 7 checks passed
@giswqs giswqs deleted the collapse branch November 8, 2025 00:39
@github-actions github-actions Bot temporarily deployed to pull request November 8, 2025 00:40 Inactive
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.

2 participants