feat: improve UX/UI on Home and TaskDetailView #3863
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#3863
Loading…
Reference in New Issue
No description provided.
Delete Branch "Jakob/frontend:feat/improve-uxui"
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?
Some UI Elements got very crazy styling ... I tried to fix them as best as I can with what was mostly there
Changes
Home
Ticket Details
.section-card
class to format sectionsshowTasks
There is so much more there ... but trying to keep it small ... more to come if appreciated
feat/improve-uxui on Home and Ticket Detailto feat: improve UX/UI on Home and Ticket Detailfeat: improve UX/UI on Home and Ticket Detailto feat: improve UX/UI on Home and TaskDetailViewHi Jakob!
Thank you for creating a PR!
I've deployed the changes of this PR on a preview environment under this URL: https://3863-feat-improve-uxui--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! Thank you for the PR.
TBH, I think this does not look not any better than what we currently have. The shadow on the task detail page is too much, elevating the details a lot. This does not really work here, as it is a completely different pattern from the rest of Vikunja where shadows are not as dominant and instead there's a more "flat-surface" look.
This has too much spacing:
What you added might look okay if the task detail page is full page, but it does not work at all in a popup:
In that case we already have a layer from the popup, the shadows create another layer on top of that. That's too many layers.
Speaking of the task detail view, what is this?:
@ -387,3 +387,3 @@
.task-project {
width: auto;
min-width: 60px; //TODO: Quickfix, find a better way to make a nice vertical alignment
What exactly is the goal here? Can you point it out with before/after screenshots?
@ -0,0 +1,8 @@
.section-card {
What is the difference between this and the existing card component?
@ -986,7 +987,6 @@ h3 .button {
}
.attachments {
margin-bottom: 0;
Why remove the spacing?
@ -291,3 +294,3 @@
:shadow="task.done"
@click="toggleTaskDone()"
class="is-outlined has-no-border"
class="is-outlined border"
I don't think this is an improvment. The button should either have no shadow or the one already set up. A 1px border looks out of place.
Hey are you still interested in this?
To be quite frank: no ... you comments didn't leave a very welcoming feeling and other projects appreciate contributions more.
The code is yours, I will not continue working on it.
Sorry if I came across as rude or something like that, that was never my intention. Looks like I was too direct.
I was a bit uneasy sending my original comments, but sent them nevertheless.
Design is subjective in a lot of places, and I guess in some parts, I just didn't like what you did (and that's nothing you did wrong). But design is also rational and objective in terms of consistency and relation of some elements to others. Going from a flat look here to a look with plenty of shadows creates an inconsistency in the overall design.
I also only criticized what I felt needs improvement. I didn't comment on the rest, which was good.