Skip to content

Conversation

arjxn-py
Copy link
Member

@arjxn-py arjxn-py commented Aug 1, 2025

Copy link
Contributor

github-actions bot commented Aug 1, 2025

Binder 👈 Launch a Binder on branch arjxn-py/jupytergis/tiles-symbology

Copy link
Contributor

github-actions bot commented Aug 1, 2025

Integration tests report: appsharing.space

@arjxn-py arjxn-py added the enhancement New feature or request label Aug 4, 2025
@arjxn-py arjxn-py changed the title [wip]: trying to get the attribute data of tiles [wip]: Enable Graduated, Categorised & Canonical Symbology on VectorTiles Aug 4, 2025
@arjxn-py arjxn-py changed the title [wip]: Enable Graduated, Categorised & Canonical Symbology on VectorTiles [wip]: Support Graduated, Categorised & Canonical Symbology on VectorTiles Aug 4, 2025
@arjxn-py arjxn-py requested review from mfisher87 and gjmooney August 5, 2025 08:12
@arjxn-py arjxn-py changed the title [wip]: Support Graduated, Categorised & Canonical Symbology on VectorTiles Support Graduated, Categorised & Canonical Symbology on VectorTiles Aug 5, 2025
@arjxn-py arjxn-py marked this pull request as ready for review August 5, 2025 08:19
Copy link
Collaborator

@gjmooney gjmooney left a comment

Choose a reason for hiding this comment

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

Nice!

Copy link
Member

@mfisher87 mfisher87 left a comment

Choose a reason for hiding this comment

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

@arjxn-py and I discussed on a call today that perhaps we should not remove stroke color from graduated/categorical vector symbology. I think it's OK if we do; we can bring it back later based on how it impacts user. If we do remove stroke, we should remove the stroke color and fill color inputs from the graduated/categorical vector symbology dialogs.

@arjxn-py arjxn-py merged commit 8582f4c into geojupyter:main Aug 6, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants