Skip to content

Conversation

@dopenguin
Copy link
Member

@dopenguin dopenguin commented Sep 4, 2025

Summary

Migrate LayerChooser plugin to @polar/polar and implement reusable components PolarCard, PolarInput and PolarInputGroup.

Instructions for local reproduction and review

Snowbox provides the LayerChooser as part of the IconMenu for layout ninenRegions and standard.

Relevant tickets, issues, et cetera

Closes #346, Blocked by #357 and #364

@dopenguin dopenguin self-assigned this Sep 4, 2025
@dopenguin dopenguin added enhancement New feature or request refactor Refactoring of previous code labels Sep 4, 2025
dopenguin and others added 20 commits September 4, 2025 18:06
…sk layers

Tooltip when a layer is disabled is missing and will follow in an upcoming commit.
dopenguin and others added 7 commits September 25, 2025 21:49
…-layer-chooser

# Conflicts:
#	examples/iceberg/App.vue
#	examples/snowbox/AnotherEmptyComponent.vue
#	examples/snowbox/index.js
#	src/core/components/PolarMap.ce.vue
#	src/core/components/layouts/NineLayout.ce.vue
#	src/core/components/layouts/StandardLayout.ce.vue
#	src/core/index.ts
#	src/core/stores/main.ts
#	src/core/types/plugin.ts
#	src/plugins/fullscreen/components/FullscreenUI.ce.vue
@dopenguin
Copy link
Member Author

@oeninghe-dataport I had to add pointer-events: all in various places in 2cb9c9b. This seems to be necessary after the recent update to @kern-ux/native.

The PR is now ready for review.

Copy link
Collaborator

@oeninghe-dataport oeninghe-dataport left a comment

Choose a reason for hiding this comment

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

🏓 @dopenguin

@oeninghe-dataport
Copy link
Collaborator

Loosely related: #386
However, not blocking for this PR, just FYI

@dopenguin
Copy link
Member Author

🏓 @dopenguin

🏓 @oeninghe-dataport

Copy link
Collaborator

@oeninghe-dataport oeninghe-dataport left a comment

Choose a reason for hiding this comment

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

🏓 @dopenguin

@dopenguin
Copy link
Member Author

🏓 @dopenguin

🏓 @oeninghe-dataport

Copy link
Collaborator

@oeninghe-dataport oeninghe-dataport left a comment

Choose a reason for hiding this comment

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

🚀 @dopenguin

@dopenguin dopenguin merged commit 1893276 into next Oct 27, 2025
3 checks passed
@dopenguin dopenguin deleted the vue3/migrate-plugin-layer-chooser branch October 27, 2025 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request refactor Refactoring of previous code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migration of LayerChooser plugin to POLAR@3

2 participants