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

dev/core#3779 add an ability to set a field as display only in formbuilder #24158

Merged
merged 1 commit into from
Sep 16, 2022

Conversation

kurund
Copy link
Contributor

@kurund kurund commented Aug 5, 2022

Overview

We should be able to set a field as display only.

After

Admin Configuration
Screenshot from 2022-08-05 12-26-41

Display on the form
Screenshot from 2022-08-05 12-26-22

@civibot
Copy link

civibot bot commented Aug 5, 2022

(Standard links)

@civibot civibot bot added the master label Aug 5, 2022
@kurund kurund changed the title add an ability to set a field as display only in formbuilder dev/core#3779 add an ability to set a field as display only in formbuilder Aug 5, 2022
@colemanw
Copy link
Member

colemanw commented Aug 5, 2022

Shouldn't "display only" be in the "Type" dropdown menu?

@kurund
Copy link
Contributor Author

kurund commented Aug 5, 2022

@colemanw

This is more like an attribute rather than Type like Required etc hence added it outside.

@kurund kurund force-pushed the display-only-fields branch from 8193d08 to aa7ac58 Compare August 8, 2022 12:02
@kurund
Copy link
Contributor Author

kurund commented Aug 8, 2022

@colemanw

As per our discussion, I have added 'DisplayOnly' type.

Screenshot from 2022-08-08 12-59-26

@colemanw colemanw merged commit 43a44bf into civicrm:master Sep 16, 2022
@colemanw
Copy link
Member

@kurund I tested this out and it works, but only for Text fields. Additional work would be needed for resolving option lists.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants