Skip to content

Specta symbology override fix#1134

Merged
martinRenou merged 2 commits intogeojupyter:mainfrom
gjmooney:sym-override-fix
Feb 24, 2026
Merged

Specta symbology override fix#1134
martinRenou merged 2 commits intogeojupyter:mainfrom
gjmooney:sym-override-fix

Conversation

@gjmooney
Copy link
Collaborator

@gjmooney gjmooney commented Feb 20, 2026

Description

This fixes a bug where the first segment’s symbology override was not applied in a Specta presentation because the viewer panel rendered before the OpenLayers map had its layers, so the override was never applied.

Depends on #1130

Checklist

  • PR has a descriptive title and content.
  • PR description contains references to any issues the PR resolves, e.g. Resolves #XXX.
  • PR has one of the labels: documentation, bug, enhancement, feature, maintenance
  • Checks are passing.
    Failing lint checks can be resolved with:
    • pre-commit run --all-files
    • jlpm run lint
  • If you wish to be cited for your contribution, CITATION.cff contains an author entry for yourself

📚 Documentation preview: https://jupytergis--1134.org.readthedocs.build/en/1134/
💡 JupyterLite preview: https://jupytergis--1134.org.readthedocs.build/en/1134/lite
💡 Specta preview: https://jupytergis--1134.org.readthedocs.build/en/1134/lite/specta

@github-actions
Copy link
Contributor

Binder 👈 Launch a Binder on branch gjmooney/jupytergis/sym-override-fix

@github-actions
Copy link
Contributor

github-actions bot commented Feb 20, 2026

Integration tests report: appsharing.space

@martinRenou martinRenou added the bug Something isn't working label Feb 20, 2026
@gjmooney gjmooney marked this pull request as ready for review February 20, 2026 15:24
@gjmooney gjmooney requested a review from martinRenou February 24, 2026 09:36
@martinRenou martinRenou merged commit f738ec3 into geojupyter:main Feb 24, 2026
17 checks passed
@gjmooney gjmooney deleted the sym-override-fix branch February 24, 2026 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants