Changed text string "Done!" to "Mark task Done!" - to avoid confusion while editing and accidentally marking a task as done when you are done editing details #1294
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
4 Participants
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: vikunja/frontend#1294
Loading…
Reference in New Issue
No description provided.
Delete Branch "(deleted):main"
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?
Changed the label "Done" when editing a task to "Mark task done" to be more specific. This will help avoid accidentally clicking on the "Done" button when you are editing a task because it appears to be the way to close the details page - but it is not.
Hi dmorlitz!
Thank you for creating a PR!
I've deployed the changes of this PR on a preview environment under this URL: https://1294-main--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!
Hi! Thanks for the PR.
Please fix the failing test: https://drone.kolaente.de/vikunja/frontend/6334/1/7
The test expects to find a button with the text "Done!" on it - because that's changed the test fails. A good solution would be to add a
v-cy="'mark-task-done-button'"
to the button and then change the selector of the test to[data-cy="mark-task-done-button"]
. That way the test would not depend on the text in the button.@ -208,3 +208,3 @@
// We can't use the api sorting here because that sorts tasks with a due date after
// ones without a due date.
this.tasks = tasks.sort((a, b) => {
this.tasks = tasks.sort((b, a) => {
Why this change?
It's written in the commit message. But seems like a different issue => should be in different pull reuqest
@ -546,3 +546,3 @@
"chooseEndDate": "Click here to set an end date",
"move": "Move task to a different list",
"done": "Done!",
"done": "Mark task done!",
The test is now failing. Can you adjust the "Can mark a task as done" test?
Edit: see @konrads comment.
@ -366,4 +366,4 @@ export default {
.list-view .task-add {
padding: 1rem 1rem 0;
}
</style>
Can you remove these changes?
We try to limit changes that are not necessary.
We want to integrate a formater (prettier). Until then we try to minize diffs to simplify merges.
@ -278,4 +278,4 @@ h3 {
.llama-cool {
margin-top: 2rem;
}
</style>
Can you remove these changes?
@dmorlitz No need to close the PR right away (did we came across as rude? That wasn't our intention)
Was not our intention at all!