25.2.5+#10
Conversation
|
Please correct the project and update this pull request by pushing a new commit. https://codecentralfarm.devexpress.devx/job/blazor-grid-show-context-menu_25.1.3+/20/console |
|
Please correct the project and update this pull request by pushing a new commit. https://codecentralfarm.devexpress.devx/job/blazor-grid-show-context-menu_25.1.3+/21/console |
|
Please correct the project and update this pull request by pushing a new commit. https://codecentralfarm.devexpress.devx/job/blazor-grid-show-context-menu_25.1.3+/22/console |
| return dataColumn.AllowGroup ?? grid.AllowGroup; | ||
| return false; | ||
| } | ||
| public static void CustomizeContextMenu(GridCustomizeContextMenuEventArgs args) { |
There was a problem hiding this comment.
Should be like this instead of the same line
if (args.Context is GridDataRowCommandContext rowContext)
AddRowItems(args, rowContext);
| } | ||
|
|
||
| private static void AddRowItems(GridCustomizeContextMenuEventArgs args, GridDataRowCommandContext rowContext) { | ||
| var update = (IGridCommandContext c, Action t) => { |
There was a problem hiding this comment.
please move it in a separate method since it is used in multiple places
|
Please correct the project and update this pull request by pushing a new commit. https://codecentralfarm.devexpress.devx/job/blazor-grid-show-context-menu_25.1.3+/24/console |
|
Please correct the project and update this pull request by pushing a new commit. https://codecentralfarm.devexpress.devx/job/blazor-grid-show-context-menu_25.1.3+/25/console |
|
Please correct the project and update this pull request by pushing a new commit. https://codecentralfarm.devexpress.devx/job/blazor-grid-show-context-menu_25.1.3+/26/console |
update example to use built-in context menu where possible (rows, columns, etc.) and custom context menu in other areas (pager, toolbar, editing row)