Skip to content
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(builder/0): add initial form builder layout #3126

Merged
merged 8 commits into from
Jan 13, 2022

Conversation

karrui
Copy link
Contributor

@karrui karrui commented Dec 2, 2021

This PR is part of the chain for form-v2/feat/builder branch.

Problem

This PR adds the initial layout of the form builder page.

Included is the sidebar tabs, and the drawer that is expanded when the tabs are clicked. No real logic is included for now.

Part of #2792

Solution

Breaking Changes

  • Yes - this PR contains breaking changes
    • Details ...
  • No - this PR is backwards compatible

Features:

  • feat: add BuilderDrawerContext for holding builder tab context
  • feat: add BuilderSidebar component to show current builder tab state
  • feat: add initial FormBuilderPage to render builder page and sidebar
  • feat: add MotionBox motion component (for allowing slideout animation of the drawer)
  • feat: init BuilderDrawer component to show current builder info
  • feat: add BuilderContent and initial FieldRowContainer component

Before & After Screenshots

AdminFormBuilderPage stories have been updated to show the new layout. Note that only desktop styling is included (responsive styling will be added in future PRs)

@karrui karrui requested review from theMogget and tshuli December 2, 2021 07:16
@karrui karrui force-pushed the form-v2/feat/builder/layout-render-0 branch from 658e700 to bf23514 Compare December 2, 2021 10:17
@karrui karrui merged commit 850a7c7 into form-v2/develop Jan 13, 2022
@karrui karrui deleted the form-v2/feat/builder/layout-render-0 branch January 13, 2022 07:11
@justynoh justynoh mentioned this pull request Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant