Skip to content

Managed #181

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

Merged
merged 2 commits into from
May 30, 2023
Merged

Managed #181

merged 2 commits into from
May 30, 2023

Conversation

progrium
Copy link
Contributor

Closes #105 focusing on making commands skip/ignore when working with a "managed" node (only search node for now). Repeating commit message since it includes command behaviors changed for QA:

  • create-field is ignored when path parent is managed
  • indent skips managed node and uses previous/above sibling or cancels
  • outdent is ignored when path parent is managed
  • move-up skips managed node and uses previous/above sibling or cancels
  • move-down skips managed node and uses next/below sibling or cancels
  • insert-child (inc newline at end of open parent) is ignored when node is managed
  • insert and insert-before is ignored when path parent is managed
  • create-reference is ignored when path parent is managed

@taramk

progrium added 2 commits May 30, 2023 11:20
- introduce objectManaged for determining if node has component that returns custom children
- create-field is ignored when path parent is managed
- indent skips managed node and uses previous/above sibling or cancels
- outdent is ignored when path parent is managed
- move-up skips managed node and uses previous/above sibling or cancels
- move-down skips managed node and uses next/below sibling or cancels
- insert-child (inc newline at end of open parent) is ignored when node is managed
- insert and insert-before is ignored when path parent is managed
- create-reference is ignored when path parent is managed
@progrium progrium merged commit b70c4cc into main May 30, 2023
@progrium progrium deleted the managed branch July 6, 2023 15:26
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.

app: add restrictions to search node
1 participant