DevExpress / devextreme-reactive

Business React components for Bootstrap and Material-UI
https://devexpress.github.io/devextreme-reactive/
Other
2.07k stars 376 forks source link

EditRecurrenceMenu doesn't show before onCommitChanges function is called #3638

Closed zgajo closed 11 months ago

zgajo commented 1 year ago

Is there an existing issue for this?

I'm using ...

React Scheduler

Current Behaviour

Use a scheduler like in the code below doesn't show Confirmation when an appointment is dragged to another time:


<Scheduler firstDayOfWeek={1} locale={locale.locale} data={schedulerData}>
              <ViewState defaultCurrentViewName="Week" />
              <EditingState onCommitChanges={commitChanges} />
              <EditRecurrenceMenu />
             {<GroupingState grouping={grouping} groupByDate={groupByDate} />}

              <DayView />

              <WeekView dayScaleCellComponent={WeekDayScaleCell} />

              <Toolbar />
              <TodayButton />
              <DateNavigator />

              <Appointments />
              {<Resources data={resources} />}
              {<IntegratedGrouping />}
              {/* <IntegratedEditing /> */}
              <ConfirmationDialog ignoreCancel />

              <AppointmentTooltip showDeleteButton showOpenButton />
              <AppointmentForm
                basicLayoutComponent={ManageAppointment}
                onVisibilityChange={(visible) => {
                  if (!visible) {
                    fetch();
                  }
                }}
              />

              <ViewSwitcher />
              {<GroupingPanel />}
              <DragDropProvider />
              <CurrentTimeIndicator
                shadePreviousCells
                shadePreviousAppointments
                updateInterval={300000} // 5 min
                indicatorComponent={Indicator}
              />
            </Scheduler>
```an 

### Expected Behaviour

Should show confirmation modal like in the example 
https://codesandbox.io/s/immutable-meadow-4pv646?file=/demo.js:279-297

### Steps to Reproduce

_No response_

### Environment

- devextreme-reactive: 4.0.3
- react: 17.0.2
- browser: Chrome
- bootstrap: 5.11.8
- react-bootstrap: none
- material-ui: none
artem-kurchenko commented 11 months ago

Hi @zgajo

The example you attached throws errors on my side: https://codesandbox.io/s/immutable-meadow-4pv646?file=/demo.js:3958-3978

Could you please modify this example to demonstrate the issue: https://codesandbox.io/s/dazzling-platform-jkc269?file=/demo.js ?

github-actions[bot] commented 11 months ago

This issue has been automatically closed because there has been no response to our request for more information from the original author. With only the information that is currently in the issue, we don't have enough information to take action. Please reach out if you have or find the answers we need so that we can investigate further.