fix: related tasks add button and task dates in read only view #1268
No reviewers
Labels
No Label
area/internal-code
changes requested
confirmed
dependencies
duplicate
good first issue
help wanted
hosting
invalid
kind/bug
kind/feature
question
wontfix
No Milestone
No project
No Assignees
3 Participants
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: vikunja/frontend#1268
Loading…
Reference in New Issue
No description provided.
Delete Branch "fix/read-only-layout"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Fixes #1052 and the "created/updated/done at" at the bottom right corner for read only views.
@ -0,0 +25,4 @@
import TaskModel from '@/models/task'
import {formatDateLong, formatDateSince} from '@/helpers/time/formatDate'
const {task} = defineProps({
As the failing test says:
*EDIT: you can however use toRefs and then use
task.value
Changed it to use
toRefs
.@ -249,2 +249,2 @@
<div class="column is-one-third action-buttons">
<a @click="$router.back()" class="is-fullwidth is-block has-text-centered mb-4" v-if="shouldShowClosePopup">
<div class="column is-one-third action-buttons" v-if="canWrite || shouldShowClosePopup">
<a @click="$router.back()" class="is-fullwidth is-block has-text-centered mb-4">
Should be a
<BaseButton>
Done.
@ -697,239 +671,239 @@ export default {
$flash-background-duration: 750ms;
.task-view {
Reduce diff.
We should separate formatting commits and changing commits.
If we mix both in the same it's hard to track changes.
Best would be to wait for #930 and then use prettier (it also formats CSS).
I've converted this back to spaces which greatly minimized the diff. There's still some diff that seems unrelated but I think it should be fine now.
Hi konrad!
Thank you for creating a PR!
I've deployed the changes of this PR on a preview environment under this URL: https://1268-fixread-only-layout--vikunja-frontend-preview.netlify.app
You can use this url to view the changes live and test them out.
You will need to manually connect this to an api running somehwere. The easiest to use is https://try.vikunja.io/.
Have a nice day!
4f752b4c25
to551ec184bf
Will merge once the ci passes.