Scheduler: fix broken resize in vertical grouping

[sjbur]

No description provided.

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Suppressed comments (1)

packages/devextreme/js/__internal/scheduler/appointments/appointment/m_appointment.ts:289

• This also disables strict snapping for horizontal appointments; all-day items are explicitly assigned direction: 'horizontal' in generate_grid_view_model.ts:87. The unequal all-day row height only disrupts vertical edge alignment, so horizontal all-day resizing should retain strict precision instead of switching to delta-based rounding. Gate this workaround on vertical direction.

    const allDayBreaksCellGrid = Boolean(this.invoke('isVerticalGroupedWorkSpace'))
      && allDayHeight > 0
      && allDayHeight !== cellHeight;

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.

Suppressed comments (1)

packages/devextreme/js/__internal/scheduler/m_subscribes.ts:204

• This new workspace dependency is missing from the SchedulerWorkSpaceLike contract in scheduler.ts (unlike getCellHeight and isVerticalGroupedWorkSpace). Because this mixin's this is effectively untyped, the omission is not caught here and future workspace implementations can satisfy the interface without providing this method. Add getAllDayHeight: () => number to that interface so this call is statically checked.

    return this.getWorkSpace().getAllDayHeight();