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

Enhancement: Add conditionally show/hide forms #20

Merged
merged 6 commits into from Jan 16, 2015

Conversation

Projects
None yet
3 participants
@ounziw
Contributor

ounziw commented Jan 8, 2015

Some forms are shown when the condition is met.

Example Usage: When I select "Other", another textarea is shown to fill in the details.

JavaScript codes are updated, following the suggestions by @vrcAlbert
http://forums.novius-os.org/en/support-forums/contributions/noviusos_form-when-select-other-another-textarea-shown-fill-the-details,feed,267.html

ounziw added some commits Jan 5, 2015

add conditions
Example Usage:
When I select "Other", another textarea is shown to fill in the details.
@jguyomard

This comment has been minimized.

Show comment
Hide comment
@jguyomard

jguyomard Jan 8, 2015

Collaborator

Thx, nice feature ! 👍

Collaborator

jguyomard commented Jan 8, 2015

Thx, nice feature ! 👍

@Foine

This comment has been minimized.

Show comment
Hide comment
@Foine

Foine Jan 8, 2015

Contributor

Thank you for this ! 👍

Contributor

Foine commented Jan 8, 2015

Thank you for this ! 👍

@ounziw

This comment has been minimized.

Show comment
Hide comment
@ounziw

ounziw Jan 12, 2015

Contributor

The reason to this commit is,
ounziw@18fe506

You will fill the "field_virtual_name" into the "field_conditional_form".
"field_virtual_name" is shown on "expert view". It is better that "field_conditional_form" is also shown on "expert view".

Contributor

ounziw commented Jan 12, 2015

The reason to this commit is,
ounziw@18fe506

You will fill the "field_virtual_name" into the "field_conditional_form".
"field_virtual_name" is shown on "expert view". It is better that "field_conditional_form" is also shown on "expert view".

@Foine

This comment has been minimized.

Show comment
Hide comment
@Foine

Foine Jan 16, 2015

Contributor

Perfect. Thank you !

Contributor

Foine commented Jan 16, 2015

Perfect. Thank you !

Foine added a commit that referenced this pull request Jan 16, 2015

@Foine Foine merged commit d489645 into novius-os:master/elche Jan 16, 2015

1 check passed

Scrutinizer 1 updated code elements
Details

@ounziw ounziw changed the title from Dev condtion2 to Enhancement: Add conditionally show/hide forms Feb 18, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment