Skip to content

Use full height space when displaying widget in linked view or sidecar#645

Closed
mfisher87 wants to merge 1 commit intogeojupyter:mainfrom
mfisher87:fix-widget-height-in-sidecar-and-linked-output
Closed

Use full height space when displaying widget in linked view or sidecar#645
mfisher87 wants to merge 1 commit intogeojupyter:mainfrom
mfisher87:fix-widget-height-in-sidecar-and-linked-output

Conversation

@mfisher87
Copy link
Member

@mfisher87 mfisher87 commented Apr 18, 2025

Description

Currently, when displaying a GISDocument widget, it will always have a fixed height of 600px. This is problematic when opening the widget in a sidecar or a "linked output" view, as the widget will either take up a small portion of the height of the container or it will extend beyond the bottom of the container and require the user to scroll.

This PR sets the height to 100% in both of those contexts.

"Linked output" view can be opened by right-clicking the gutter space to the left of a widget and selecting "Create new view for cell output".

Sidecar view can be opened with jupyterlab-sidecar.

Thanks @SylvainCorlay for pointing me in the right direction 🚀

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--645.org.readthedocs.build/en/645/
💡 JupyterLite preview: https://jupytergis--645.org.readthedocs.build/en/645/lite

Linked view can be opened by right-clicking the gutter space to the left
of a widget and selecting "Create new view for cell output".

Sidecar view can be opened with
[jupyterlab-sidecar](https://github.com/jupyter-widgets/jupyterlab-sidecar).
@mfisher87 mfisher87 added the enhancement New feature or request label Apr 18, 2025
@github-actions
Copy link
Contributor

Binder 👈 Launch a Binder on branch mfisher87/jupytergis/fix-widget-height-in-sidecar-and-linked-output

@arjxn-py
Copy link
Member

Looks like a duplicate of #643 :)

@github-actions
Copy link
Contributor

Integration tests report: appsharing.space

@mfisher87 mfisher87 closed this Apr 18, 2025
@mfisher87
Copy link
Member Author

Yep! I didn't realize Sylvain had opened that PR, still waking up 😆

@mfisher87 mfisher87 deleted the fix-widget-height-in-sidecar-and-linked-output branch April 18, 2025 15:03
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