Delete key is still able to delete the read-only cell's content.

[rattagarn]

I set the cell with CellFormat.IsReadOnly to true. This works properly and prevents the user to modify the content in the cell. Unfortunately it cannot prevent the user from delete the content in that cell with 'Delete' key.

I saw this code in Datasheet.razor.cs file. It seems this key cannot detect IsReadOnly in CellFormat. Do I understand correctly?

ShortcutManager.Register(["Delete", "Backspace"], KeyboardModifiers.Any, _ => _sheet.Commands.ExecuteCommand(new ClearCellsCommand(_sheet.Selection.Regions)), _ => _sheet.Selection.Regions.Any() && !_sheet.Editor.IsEditing);

[anmcgrath]

Yes you are correct, this is a bug. Seems like you can also paste into it.

[deanparker-web]

I use the following and it works for me. The idea is to cancel the edit mode and it will prevent the user from editing the cells.

You can use more logic by viewing the post: https://github.com/anmcgrath/BlazorDatasheet/discussions/103#discussion-7222702

razor page
<Datasheet Sheet="sheet" FrozenTopCount="0" FrozenLeftCount="1" StickyHeaders="true" />

razor.cs page
The code is

    protected override async Task OnInitializedAsync()
    {
        await base.OnInitializedAsync();
        await LoadTableData();
        StateHasChanged();
    }

    // If the user clicks on any column it is read only, except column 12.
    // Logic: Cancel Edit (true/false) if the cell is not equal to 12.
    private async Task LoadTableData()
    {
        sheet.Editor.BeforeCellEdit += (sender, args) => { args.CancelEdit = !(args.Cell.Col == 12); };
        ....
    }

[rattagarn]

Thank you @deanparker-web for this information. I will use this method.

[rattagarn]

Hi @deanparker-web

Actually it won't work when the user just selected the cell (the cell hasn't yet changed to edit mode) and press 'Delete' key.
Normally when the cell is read-only, user cannot enter the edit mode in that cell.

[janek49]

We managed to work around this by catching the BeforeCommand event and cancelling it if it's pasting/deleting into a readonly cell. This works great.

[rattagarn]

Can you show your code that prevent the pasting/deleting command?
Currently I modify this. I feel it may not be the right way to solve the issue.

modify ClearCellsCommand.cs

    public bool Execute(Sheet sheet)
    {   
        //beginning of code: prevent delete key 
        foreach (var r in _regions)
        {
            for (int x = r.Left; x <= r.Right; x++)
            {
                for (int y = r.Top; y <= r.Bottom; y++)
                {
                    bool? readOnly = sheet.Cells[y, x].Format.IsReadOnly;
                    if (readOnly??false)
                        return false;
                }
            }            
        }
       //end of code
        _restoreData = sheet.Cells.ClearCellsImpl(_regions);
        return true;
    }

[parker-dean]

A version I got working is using the following logic.

        if (editor.Sheet.GetFormat(row, col).IsReadOnly is not null)
        {
            if ((bool)editor.Sheet.GetFormat(row, col).IsReadOnly)
            {
                // data equals 1 row of User data, Using int col convert to column name and get value.
                editor.Sheet.Cells[row, col].Value = data.GetType().GetProperty(_cols[col])?.GetValue(data, null);
                return;
            }
        }

[anmcgrath]

Hi all, thanks for the input. See PR #180 which should fix this issue.