Skip to content

Add Story mode presentation toggle button to toolbar#1056

Merged
martinRenou merged 5 commits intogeojupyter:mainfrom
gjmooney:add-presentation-toggle
Jan 27, 2026
Merged

Add Story mode presentation toggle button to toolbar#1056
martinRenou merged 5 commits intogeojupyter:mainfrom
gjmooney:add-presentation-toggle

Conversation

@gjmooney
Copy link
Collaborator

@gjmooney gjmooney commented Jan 5, 2026

Description

Adds a story mode presentation toggle button to the toolbar

image

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

@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2026

Binder 👈 Launch a Binder on branch gjmooney/jupytergis/add-presentation-toggle

@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2026

Integration tests report: appsharing.space

@gjmooney gjmooney force-pushed the add-presentation-toggle branch from 1b592f9 to 7cef6c1 Compare January 6, 2026 08:21
@gjmooney gjmooney added the bug Something isn't working label Jan 6, 2026
@martinRenou martinRenou reopened this Jan 16, 2026
[CommandIDs.temporalController]: { icon: clockIcon },
[CommandIDs.addMarker]: { icon: markerIcon },
[CommandIDs.addStorySegment]: { iconClass: 'fa fa-link' },
[CommandIDs.toggleStoryPresentationMode]: { iconClass: 'fa fa-dragon' },
Copy link
Member

Choose a reason for hiding this comment

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

? dragon? 😆

May I suggest this instead?

Suggested change
[CommandIDs.toggleStoryPresentationMode]: { iconClass: 'fa fa-dragon' },
[CommandIDs.toggleStoryPresentationMode]: { iconClass: 'fa fa-book-open-reader' },

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Dragons go in stories! It makes sense! 🤣

@gjmooney
Copy link
Collaborator Author

bot please update snapshots

@gjmooney gjmooney force-pushed the add-presentation-toggle branch from 3d74b33 to 8711aee Compare January 19, 2026 17:09
@gjmooney
Copy link
Collaborator Author

bot please update snapshots

1 similar comment
@gjmooney
Copy link
Collaborator Author

bot please update snapshots

@martinRenou martinRenou force-pushed the add-presentation-toggle branch from 9d57dcd to c9a64bb Compare January 27, 2026 09:39
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 21aad1e into geojupyter:main Jan 27, 2026
15 checks passed
gjmooney added a commit to gjmooney/jupytergis that referenced this pull request Feb 18, 2026
* Add Story mode presentation toggle button to toolbar

* Lint

* Update story toggle icon

* Update Playwright Snapshots

* Disable toggle presentation button when no segments & change icon

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@gjmooney gjmooney deleted the add-presentation-toggle branch February 24, 2026 11:56
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