-
Notifications
You must be signed in to change notification settings - Fork 377
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: form dialog extension #4296
Conversation
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.
submitting comments so far. this diff will take awhile to go through
Composer/packages/form-dialogs/src/components/common/CommandBarUpload.tsx
Show resolved
Hide resolved
Composer/packages/form-dialogs/src/components/property/PropertyTypeSelector.tsx
Show resolved
Hide resolved
I think this PR won't affect the current UI as it isn't wired into Composer top-level, correct? When we get to the PR does does make this show up, let's be sure it is behind the feature switching mechanism. |
@cwhitten I'm going to merge this PR since as Geoff has said it too, it will not affect any part of the Composer. I'm open to further comments and suggestions and will PR them separately if there were any. |
* Form dialogs init * new design * UX comments * conflicts resolved * yarn * click outside to dismiss expanded * Fix yarn.lock * PR comments * fix demo Co-authored-by: Soroush <sorgh@microsoft.com>
Description
Form dialog extension UI
This PR adds the UI for creating form dialog schema. Once the project tree is ready, form dialog will be integrated into the client web app.
Task Item
closes #4145
closes #4146
Screenshots
(demo)