[Feature] view change & scrollTo improvement

[n1crack]

Feature Title

return a promise after view change

Feature Description

After changing the current view to "week" view, I should be able to scroll to an event.

Use Case

If we return a Promise object with the switch method, it can be very useful for others, they can perform other desired tasks when the operation is finished.

like :

const goToEvent = async () => {
  await currentCal.value.view.switch('week', new Date());
  currentCal.value.view.scrollToCurrentTime();
}; 

https://stackblitz.com/edit/vuecal5-events-qhepnrh4?file=src%2FApp.vue

[antoniandre]

Hi @n1crack,

Thanks for the details and repro link.
I agree this would be a convenient and valuable feature. However, it’s currently not technically feasible to implement in a reliable way.

Even though I could expose all view methods as async, there’s a complex reactivity chain involved in redrawing cells and updating related state when the view changes. This makes it impossible to await the full completion of these updates directly.

The most reliable solution is to listen for the view-change event. That’s the best hook to use for triggering actions like scrolling.

Another approach is to use a setTimeout that waits for the view transition: default --vuecal-transition-duration: 0.25s;, plus a minimal delay (as short as possible); a total of 300ms usually suffices.

I'm open to discuss other suggestions, as long as it's an improvement to the codebase.

[joweste]

Ok, but scrollToTime sitill is bugged. You need take account "timeFrom.value" and subtract it. Otherwise, when it is different from 00:00 scrollToTime will be positioned on wrong place.
I said it in [Bug] ScrollToTime fix proposal #46