Skip to content

[stable35] fix(gantt): keep exact dates when dragging or clicking bars - #8572

Merged
mejo- merged 1 commit into
stable35from
backport/8570/stable35
Oct 20, 2026
Merged

mejo- merged 1 commit into
stable35from
backport/8570/stable35

Conversation

@backportbot

@backportbot backportbot Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Backport of #8570

Warning, This backport's changes differ from the original and might be incomplete 鈿狅笍

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot
backportbot Bot requested a review from mejo- October 20, 2026 11:15
@mejo-
mejo- force-pushed the backport/8570/stable35 branch from f3f53b3 to 59a4726 Compare October 20, 2026 11:36
@mejo-
mejo- marked this pull request as ready for review October 20, 2026 11:36
frappe-gantt converts the bar position back to dates in whole units,
truncating fractions. In the month view this snapped dates to the turn
of the month, and bars shorter than a month ended one second before
they started. The week and day views lost the time of day or minutes.
The conversion also ran on any mouse movement while a bar was pressed,
so just clicking a bar to open the card changed its dates.

Take the card dates from the store when a drag starts and shift them by
the dragged distance instead. Full days are added on the calendar so the
time of day is kept across DST changes. Clicking a bar without moving it
no longer saves anything.

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: ClaudeCode:claude-opus-5-5
@mejo-
mejo- force-pushed the backport/8570/stable35 branch from 59a4726 to 069b73f Compare October 20, 2026 12:27
@mejo-
mejo- merged commit a2b185e into stable35 Oct 20, 2026
38 checks passed
@mejo-
mejo- deleted the backport/8570/stable35 branch October 20, 2026 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant