#246 No list colors in overview / next week / next month

Open
opened 2 months ago by Bouni · 8 comments
Bouni commented 2 months ago

Hi,

I assigned different colors to my Namespaces but in the overview no colors are used at all.

I would really like to have a color indication so that I can see more quickly which namespaces my todos are in.

Hi, I assigned different colors to my Namespaces but in the overview no colors are used at all. I would really like to have a color indication so that I can see more quickly which namespaces my todos are in.
konrad commented 2 months ago
Owner

Sounds reasonable. What do you think would be a good way to show the colours in the overview? Maybe like it is done in the menu?

Do you want to send a pr?

Sounds reasonable. What do you think would be a good way to show the colours in the overview? Maybe like it is done in the menu? Do you want to send a pr?
Bouni commented 2 months ago
Poster

Yes, a round circle with the right color in front of the checkbox would be nice!
Also that would give a consistent UI feeling in my opinion.

I can try to submit a PR, but my Vue knowledge is very limited to be honest :-)
Let me take a look at it and I'll let you know if a PR is feasible for me

Yes, a round circle with the right color in front of the checkbox would be nice! Also that would give a consistent UI feeling in my opinion. I can try to submit a PR, but my Vue knowledge is very limited to be honest :-) Let me take a look at it and I'll let you know if a PR is feasible for me
Bouni commented 2 months ago
Poster

Correct me if I'm totally wrong, but a span like this:

https://kolaente.dev/vikunja/frontend/src/branch/master/src/App.vue#L188-L192

would go between this div and the fancycheckbox here:

https://kolaente.dev/vikunja/frontend/src/branch/master/src/components/tasks/partials/singleTaskInList.vue#L2-L3

but with not with n.hexColor
I think I would need to get the right color with something like this:

${$store.getters['lists/getListById'](task.listId).hexColor}

right?

Correct me if I'm totally wrong, but a span like this: https://kolaente.dev/vikunja/frontend/src/branch/master/src/App.vue#L188-L192 would go between this div and the fancycheckbox here: https://kolaente.dev/vikunja/frontend/src/branch/master/src/components/tasks/partials/singleTaskInList.vue#L2-L3 but with not with `n.hexColor` I think I would need to get the right color with something like this: `${$store.getters['lists/getListById'](task.listId).hexColor}` right?
konrad commented 2 months ago
Owner

Just to make sure I understood you correctly: You want to show the color of the list the task belongs to in the task overview? Or the namespace color?

The namespace color does not really make sense imho and then there's also the color of the task itself. We somehow need to differentiate the two to avoid confusion for users.

In general your code starting point would be correct however

  • You need to make sure the color span is only shown if there is a color
  • You would also need to make sure the color is only shown in the overview and not in the task list per list as that would probably create confusion for users.
Just to make sure I understood you correctly: You want to show the color of the list the task belongs to in the task overview? Or the namespace color? The namespace color does not really make sense imho and then there's also the color of the task itself. We somehow need to differentiate the two to avoid confusion for users. In general your code starting point would be correct however * You need to make sure the color span is only shown if there is a color * You would also need to make sure the color is only shown in the overview and not in the task list per list as that would probably create confusion for users.
Bouni commented 2 months ago
Poster

I mean like in the attached picture, maybe it makes sense to put the color bubble in between checkbox and listname.

I just now realized that I mixed namespace and list in my issue, sorry for that 😕

I mean like in the attached picture, maybe it makes sense to put the color bubble in between checkbox and listname. I just now realized that I mixed namespace and list in my issue, sorry for that 😕
konrad changed title from No namespace colors in overview / next week / next month to No list colors in overview / next week / next month 2 months ago
konrad commented 2 months ago
Owner

Ah, now I understand. I thought you were talking about the namespace overview 🙂
I updated the title of this issue accordingly.

Now that I understand what you meant, I think it would be better to put the color bubble next to the name of the list in that task, that way we could avoid confusion with task colors.

Ah, now I understand. I thought you were talking about the namespace overview 🙂 I updated the title of this issue accordingly. Now that I understand what you meant, I think it would be better to put the color bubble next to the name of the list in that task, that way we could avoid confusion with task colors.
Bouni commented 2 months ago
Poster

I tried a bit to achive this but couldn't manage to get it to work 😞
Is there any chance that you realize this feature request?

I tried a bit to achive this but couldn't manage to get it to work 😞 Is there any chance that you realize this feature request?
konrad commented 2 months ago
Owner

What are you struggeling with?

What are you struggeling with?
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.