Scheduler: fix recurrence appointment width in month timeline for virtual scrolling (T1258030) (#30508)

Co-authored-by: Vladimir Bushmanov <[email protected]>

Author: Ambrozy

e2e/testcafe-devextreme/tests/scheduler/common/virtualScrolling/T1258030.ts

@@ -0,0 +1,38 @@
+import Scheduler from 'devextreme-testcafe-models/scheduler';
+import { createScreenshotsComparer } from 'devextreme-screenshot-comparer';
+import { createWidget } from '../../../../helpers/createWidget';
+import url from '../../../../helpers/getPageUrl';
+import { scrollTo } from '../../helpers/utils';
+
+fixture.disablePageReloads`Scheduler: Virtual scrolling`
+  .page(url(__dirname, '../../../container.html'));
+
+test('it should render recurrence appointment with correct width in month timeline view for virtual scrolling', async (t) => {
+  const { takeScreenshot, compareResults } = createScreenshotsComparer(t);
+
+  const scheduler = new Scheduler('#container');
+
+  await scrollTo(3000, 0);
+  await t.expect(await takeScreenshot('virtual_scroll_timeline_3000.png', scheduler.workSpace)).ok();
+
+  await t
+    .expect(compareResults.isValid())
+    .ok(compareResults.errorMessages());
+}).before(async () => {
+  await createWidget(
+    'dxScheduler',
+    {
+      height: 300,
+      currentView: 'timelineMonth',
+      views: ['timelineMonth'],
+      currentDate: new Date(2024, 9, 1),
+      dataSource: [{
+        text: 'appointment',
+        startDate: new Date(2024, 9, 1),
+        endDate: new Date(2024, 9, 2),
+        recurrenceRule: 'FREQ=DAILY',
+      }],
+      scrolling: { mode: 'virtual' },
+    },
+  );
+});

e2e/testcafe-devextreme/tests/scheduler/common/virtualScrolling/etalons/virtual_scroll_timeline_3000.png

@@ -222,16 +222,8 @@ class VirtualStrategy extends BaseStrategy {
   isVirtualScrolling = true;
 
   calculateCellPositions(groupIndices, isAllDayRowAppointment, isRecurrentAppointment) {
-    const appointments = isAllDayRowAppointment
-      ? this.appointments
-      : this.appointments.filter(({ source, startDate, endDate }) => this.viewDataProvider.isGroupIntersectDateInterval(
-        source.groupIndex,
-        startDate,
-        endDate,
-      ));
-
     if (isRecurrentAppointment) {
-      return this.createRecurrentAppointmentInfos(appointments, isAllDayRowAppointment);
+      return this.createRecurrentAppointmentInfos(this.appointments, isAllDayRowAppointment);
     }
 
     return super.calculateCellPositions(groupIndices, isAllDayRowAppointment, isRecurrentAppointment);

packages/devextreme/js/__internal/scheduler/appointments/m_cell_position_calculator.ts

@@ -657,10 +657,17 @@ export class AppointmentSettingsGenerator {
       itemGroupIndices,
       isRecurrent,
     } = this._generateDateSettings();
+    const { isVirtualScrolling, viewDataProvider } = this.options;
+    const filteredDateSettings = this.isAllDayRowAppointment || !isVirtualScrolling
+      ? dateSettings
+      : dateSettings.filter(({ source, startDate, endDate }) => viewDataProvider.isGroupIntersectDateInterval(
+        source.groupIndex,
+        startDate,
+        endDate,
+      ));
 
-    const cellPositions = this._calculateCellPositions(dateSettings, itemGroupIndices);
-
-    const result = this._prepareAppointmentInfos(dateSettings, cellPositions, isRecurrent);
+    const cellPositions = this._calculateCellPositions(filteredDateSettings, itemGroupIndices);
+    const result = this._prepareAppointmentInfos(filteredDateSettings, cellPositions, isRecurrent);
 
     return result;
   }