Skip to content

Conversation

gjmooney
Copy link
Collaborator

@gjmooney gjmooney commented Jun 10, 2025

Description

Creates some shared components, these will be used in the STAC browser.

These are based on ShadCN components and the work done in shadcn-css, adapted to use Jupyter CSS variables.

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

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

@gjmooney gjmooney added the enhancement New feature or request label Jun 10, 2025
Copy link
Contributor

Binder 👈 Launch a Binder on branch gjmooney/jupytergis/create_shared_components

@gjmooney gjmooney requested a review from martinRenou June 10, 2025 10:11
Copy link
Contributor

github-actions bot commented Jun 10, 2025

Integration tests report: appsharing.space

Copy link
Member

@martinRenou martinRenou left a comment

Choose a reason for hiding this comment

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

Thanks!

@martinRenou martinRenou merged commit a6f6085 into geojupyter:main Jun 11, 2025
14 checks passed
elifsu-simula pushed a commit to elifsu-simula/jupytergis that referenced this pull request Jun 12, 2025
* Create some shared components

* Add tabs; Add CSS

* Button CSS

* Use newer button

* Add components for date picker

* Add pagination component

* Iterate on paginator

* Tiny tweaks

* Use the correct package.json

* lint

* Woops

* Use jupyter css vars

* Add sr-only class

* Lint

* Clean up
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.

2 participants