Skip to content

fix(table): respect cellRender outside edit mode - #3266

Open
guomengjia618-dot wants to merge 1 commit into
x-extends:v3from
guomengjia618-dot:agent/fix-cell-render-priority
Open

fix(table): respect cellRender outside edit mode#3266
guomengjia618-dot wants to merge 1 commit into
x-extends:v3from
guomengjia618-dot:agent/fix-cell-render-priority

Conversation

@guomengjia618-dot

Copy link
Copy Markdown

What changed

  • Prefer cellRender when rendering a cell outside edit mode.
  • Keep editRender as the fallback when no cellRender is configured.
  • Pass the matching renderer type (cell or edit) to the selected renderer.

Why

When a column defines both cellRender and editRender, the non-edit path previously selected editRender first. As a result, cellRender was never used, even when the cell was not being edited.

This keeps the existing behavior for columns that only define editRender, while allowing columns with both renderers to use cellRender for display and editRender during editing.

Fixes #3258

Validation

  • eslint packages/table/src/cell.ts passed.
  • npm run lib passed (library bundle and module builds).
  • Full repository lint/build still reports pre-existing lint errors in gulpfile.js and example Vue files; none are in the changed file.

@guomengjia618-dot
guomengjia618-dot marked this pull request as ready for review August 12, 2026 13:21
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.

1 participant