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

Improve Condition Property Editor UI #667

Open
andrewtelnov opened this issue Feb 14, 2020 · 1 comment
Open

Improve Condition Property Editor UI #667

andrewtelnov opened this issue Feb 14, 2020 · 1 comment
Labels
Milestone

Comments

@andrewtelnov
Copy link
Member

@andrewtelnov andrewtelnov commented Feb 14, 2020

End-users have hard time to understand how to edit visibleIf, enableIf and other expression/condition properties.
We need to make text editor invisible by default, show it in separate tab and show all conditions in grid view:

question name operator question value remove button
or /and question name operator question value remove button
or /and question name operator question value remove button
Add Condition button

We need to show text editor, only for advanced users or if we could not parse expression and show it in the table view.

@andrewtelnov andrewtelnov added this to the v1.5.9 milestone Feb 14, 2020
@andrewtelnov andrewtelnov modified the milestones: v1.5.9, v1.5.10 Feb 14, 2020
andrewtelnov added a commit to surveyjs/survey-library that referenced this issue Feb 14, 2020
andrewtelnov added a commit to surveyjs/survey-library that referenced this issue Feb 15, 2020
andrewtelnov added a commit that referenced this issue Feb 18, 2020
@andrewtelnov

This comment has been minimized.

Copy link
Member Author

@andrewtelnov andrewtelnov commented Feb 18, 2020

Add ability to disable expression editing in text editor, only visual.
Text editor is shown, if expression is failed to shown in visual editor

/**
* You need to set this property to false to allow your users
* build expressions visually only, without editing them in text editor.
*/
creator.allowEditExpressionsInTextEditor = true;
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.