vikunja
/
frontend
Archived
6
39
Fork 49
Code Issues 1 Pull Requests 4 Projects Releases 26 Activity

feat: replace our home-grown gantt implementation with ganttastic #2180

Merged
konrad merged 78 commits from feature/ganttastic into main 2022-10-27 16:03:27 +00:00
Conversation 41 Commits 78 Files Changed 48 +5 -4
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 3eacc0754f - Show all commits

9
src/components/tasks/gantt-chart.vue
View File

@ -118,6 +118,7 @@ function transformTaskToGanttBar(t: TaskModel) {
color: t.startDate ? (colorIsDark(t.getHexColor()) ? black : 'white') : black,
backgroundColor: t.startDate ? t.getHexColor() : 'var(--grey-100)',
border: t.startDate ? '' : '2px dashed var(--grey-300)',
'text-decoration': t.done ? 'line-through' : null,
},
},
}]
@ -137,9 +138,9 @@ async function loadTasks() {
const params = {
sort_by: ['start_date', 'done', 'id'],
order_by: ['asc', 'asc', 'desc'],
filter_by: ['done', 'start_date', 'start_date'],
filter_comparator: ['equals', 'greater_equals', 'less_equals'],
filter_value: ['false', props.dateFrom, props.dateTo],
filter_by: ['start_date', 'start_date'],
filter_comparator: ['greater_equals', 'less_equals'],
filter_value: [props.dateFrom, props.dateTo],
filter_concat: 'and',
konrad marked this conversation as resolved Outdated
dpschen commented 2022-07-21 23:00:42 +00:00
Outdated
Review
Copy Link

define types

define types
konrad commented 2022-08-02 12:03:15 +00:00
Outdated
Review
Copy Link

Done.

Done.
👍 1
filter_include_nulls: true,
}
@ -175,7 +176,7 @@ async function updateTask(e) {
const r = await taskService.update(task)
konrad marked this conversation as resolved Outdated
dpschen commented 2022-07-21 22:44:36 +00:00
Outdated
Review
Copy Link

This and the three lines below should be combined to one watcher that trigger immediately.

loadTasks should accept these three as params, so that it's clear that these are needed to reload.

Something like:

watchEffect(() => loadTasks({
	dateTo: props.dateTo,
    dateFrom: props.dateFrom,
    showTasksWithoutDates: props.showTasksWithoutDates,
})
This and the three lines below should be combined to one watcher that trigger immediately. `loadTasks` should accept these three as params, so that it's clear that these are needed to reload. Something like: ``` watchEffect(() => loadTasks({ dateTo: props.dateTo, dateFrom: props.dateFrom, showTasksWithoutDates: props.showTasksWithoutDates, }) ```
konrad commented 2022-08-02 12:03:29 +00:00
Outdated
Review
Copy Link

Done.

Done.
👍 1
// TODO: Loading animation
for (const i in ganttBars.value) {
if(ganttBars.value[i][0].ganttBarConfig.id === task.id) {
if (ganttBars.value[i][0].ganttBarConfig.id === task.id) {
ganttBars.value[i] = transformTaskToGanttBar(r)
}
}