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

Computed Column #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Computed Column #36

wants to merge 1 commit into from

Conversation

nckenn
Copy link

@nckenn nckenn commented Jun 27, 2017

Make computed field not editable

Make computed field not editable
@ejbeaty
Copy link
Owner

ejbeaty commented Jul 17, 2017

Hi @nckenn can you help me better understand the reason for this change? Why not just omit the column index from the "columns" property so that it is ignored by CellEdit ?

@nckenn
Copy link
Author

nckenn commented Jul 18, 2017

Hi @ejbeaty , i know you have a property "columns" defining the columns that you want to be editable, and i know i can use that so it will not throw any errors. But by making computed field not editable automatically is annoying, why not add this feature so the user will have a less effort when configuring celledit. Thank you.

@Elvino12
Copy link

Elvino12 commented Apr 6, 2022

Didi some one fix this error, I´m with the same issue

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.

3 participants