-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding ODT example to storybook and fixing variorum bugs #117
Conversation
Adds ODT as a test case for variorum and polygonal shape features
Improved variorum navigation and fixed folio name collision bug
Added storybook workflow
Added storybook back
Update: Now also addresses Issue #118 by removing the blue highlighting from the "current folio" in grid mode. |
Added touchscreen functionality for resizing
Further update: Now includes a fix to the resizing functionality of the split pane view so that it now works with touchscreen devices as well as mouse clicks. |
Improving the handling of component state changes
…oncrafter into RB-state-change-fix
…oftware/editioncrafter into RB-state-change-fix
Improve handling of component props updating
Further update: Now also includes a trigger to reload the document data in the case that the component props change without the user navigating to a new page. (See PR #123 ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! 🚀
In this PR
folioID
were not being stored properly in the component state: when in variorum mode, folios are now stored internally with the ID[documentID]_[folioID]
to avoid collisions;viewportState
function to track the associated document of the currently selected folio, so that if you navigate back to the grid view from a folio in variorum mode, the document that folio came from will be automatically selected;