feat: add Gantt chart view for boards#7812
Conversation
aparcar
requested review from
elzody,
grnd-alt and
luka-nextcloud
as code owners
grnd-alt
left a comment
grnd-alt
left a comment
There was a problem hiding this comment.
works thanks a lot for this contribution. But there is some room for improvement. also darkmode looks like this:
aparcar
force-pushed
the
gantt
branch
2 times, most recently
from
1f25f7f to
e1f6cb4
Compare
|
@grnd-alt I ran Claude again and worked on the fixes. Since this is mostly AI generating (somewhat usable) things, I understand that this quickly starts to waste your time. In other words, please feel free to close this PR until someone comes along with actual web dev knowledge to create a proper implementation. |
|
Hello there, We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process. Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6 Thank you for contributing to Nextcloud and we hope to hear from you soon! (If you believe you should not receive this message, you can add yourself to the blocklist.) |
grnd-alt
force-pushed
the
gantt
branch
4 times, most recently
from
683a88f to
3d3a95a
Compare
mejo-
left a comment
mejo-
left a comment
There was a problem hiding this comment.
Overall a great addition ❤️
I tested a bit, skimmed through the code changes and made some comments.
|
|
||
| <div v-if="ganttTasks.length > 0" ref="ganttContainer" class="gantt-chart" /> | ||
|
|
||
| <NcEmptyContent v-if="!ganttTasks.length && !undatedCards.length"> |
There was a problem hiding this comment.
For an empty board, it might be better to switch back to the kanban view, no? When I create a list using this interface, I cannot really do anything with it afterwards, as the gantt view doesn't allow to add another list or cards anyway.
grnd-alt
force-pushed
the
gantt
branch
3 times, most recently
from
5e4cd63 to
c7854da
Compare
|
Discussing this with colleagues, they asked for direct links to the gantt or deck view. Could this be added to the url handling? |
|
@aparcar we can think about it as a follow up, but I'd prefer to get this pr done for now so we can have it ready in 34. |
mejo-
left a comment
mejo-
left a comment
There was a problem hiding this comment.
Tested again, and works well.
As written in chat, moving a card on the Gantt diagram results in two PUT .../api/v1.0/cards/<cardId> requests, the first with startdate: null and the second with a value in startdate.
I guess there's a flaw somewhere and the first request should not be sent.
Apart from that it looks good. Approving already so you don't need to ping me again after fixing this final issue 😉
Add a timeline/Gantt view as an alternative to the kanban board view, allowing users to visualize card schedules across time using frappe-gantt. - Add GanttView component with Day/Week/Month view modes - Add Kanban/Gantt view toggle in board controls - Store view mode preference in localStorage via Vuex - Stack-based color coding with legend and undated cards section - Drag-and-drop support for rescheduling cards - Auto-fit column width to fill container on wider views - Add frappe-gantt dependency and webpack resolve alias for its CSS Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Paul Spooren <mail@aparcar.org>
Signed-off-by: grnd-alt <git@belakkaf.net>
Signed-off-by: grnd-alt <git@belakkaf.net>
Signed-off-by: grnd-alt <git@belakkaf.net>
Signed-off-by: grnd-alt <git@belakkaf.net>
|
Cool! Thanks for making this happen |
3 participants
Summary
Add a timeline/Gantt view as an alternative to the kanban board view, allowing users to visualize card schedules across time using frappe-gantt.
This is inspired by https://github.com/nextcloud-community/ncgantt
TODO
Checklist