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: Unify CRUD in dialog diff #2965

Closed
wants to merge 21 commits into from

Conversation

zhixzhan
Copy link
Contributor

@zhixzhan zhixzhan commented May 9, 2020

Description

Unified Adaptive dialog CRUD operations checklist

Visual Editor

  • DELETE action via '...' flyout
  • ADD action via '+' flyout
  • PASTE action via '+' flyout

Form Editor

  • UPDATE action
  • UPDATE trigger
  • UPDATE dialog

Shell - Toolbar

  • COPY action selection
  • CUT action selection
  • MOVE action selection + ADD target trigger via modal
  • DELETE action selection

Shell - ProjectTree

  • ADD trigger via 'New Trigger...' modal
  • ADD dialog via 'New Dialog...' modal
  • DELETE trigger via '...' flyout
  • DELETE dialog via '...' flyout

Resource types X Operations

  • Dialog x Add | Update | Delete
  • Trigger x Add | Update | Delete
  • Action x Add | Update | Delete | Copy | Paste | Move

Task Item

refs #2699
refs #2699

Screenshots

Untitled1

@zhixzhan zhixzhan deleted the centralize-action2 branch May 9, 2020 08:04
@coveralls
Copy link

Coverage Status

Coverage remained the same at 0.0% when pulling 713111b on zhixzhan:centralize-action2 into 4c69132 on microsoft:master.

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.

2 participants