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

Add support for For/If in FormDescriptionEditor #2322

Merged
merged 1 commit into from Sep 4, 2023

Conversation

pcdavid
Copy link
Member

@pcdavid pcdavid commented Aug 30, 2023

  • [2236] Add ADR for dynamic widgets support in Forms
  • [2236] Add ADR about improved widget id computation
  • [2236] Update View.Form metamodel
  • [2236] Add support for For & If in Forms
  • [2236] Add tests
  • [2236] Add support for For/If in FormDescriptionEditor

@pcdavid pcdavid force-pushed the pcd/enh/dynamic-widgets-fde branch 6 times, most recently from c9ffe67 to 40dc930 Compare September 1, 2023 13:36
@pcdavid pcdavid marked this pull request as ready for review September 1, 2023 13:36
@pcdavid pcdavid added this to the 2023.10.0 milestone Sep 1, 2023
@pcdavid pcdavid linked an issue Sep 1, 2023 that may be closed by this pull request
@pcdavid pcdavid force-pushed the pcd/enh/dynamic-widgets-fde branch 3 times, most recently from 2b9893b to 832e717 Compare September 1, 2023 15:23
@@ -0,0 +1,37 @@
/*******************************************************************************
* Copyright (c) 2022, 2023 Obeo.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a new file, it should be 2023

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

/**
* The properties of the flexbox container component.
*
* @author arichard
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrong author

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

Bug: #2236
Signed-off-by: Pierre-Charles David <pierre-charles.david@obeo.fr>
@pcdavid pcdavid merged commit f8b0686 into master Sep 4, 2023
4 checks passed
@pcdavid pcdavid deleted the pcd/enh/dynamic-widgets-fde branch September 4, 2023 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add initial support for EEF's DynamicFor/If to Forms
2 participants