-
Notifications
You must be signed in to change notification settings - Fork 43
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
(feat) O3-3112: Re-instate the Form collapse #234
Conversation
import { SessionMode } from '../types'; | ||
import { useEffect, useState } from 'react'; | ||
|
||
export function useFormCollapse(sessionMode: SessionMode) { |
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.
This looks over engineered. I would opt for passing this state to the FormEngine
component within the esm-form-engine-app
and use it to toggle the sections
@@ -18,6 +18,7 @@ function Sidebar({ | |||
setValues, | |||
allowUnspecifiedAll, | |||
defaultPage, | |||
hideFormCollapseToggle, |
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.
Is there a particular reason u added this to the sidebar? there no sections in the sidebar (side navigation)
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.
I added this to be called when the the form is cancelled, so as to remove the toggle from workspace header.
a55c915
to
e9f979a
Compare
Requirements
Summary
The PR re-instates the form collapse functionality, and this depends on this (feat) O3-3112: Add form collapse toggle to the
esm-form-engine-app
Screenshots
form-collapse.mp4
Related Issue
O3-3112
Other