search

pgorod / power-todoist-card

Improved Todoist card for Home Assistant Lovelace UI.
MIT License
20 stars 7 forks source link

Sub task handling #12

Open PavelD opened 2 months ago

PavelD commented 2 months ago

POC of sub task handling. Result of #11 .

On my test environment I'm not able to see labels. The background color is not propagated even if the color defaults to black :-( So I'm not sure if it's just my issue or it's some bug in display.

I can see labels only if I select them.

Ready for comments and advises as I'm not JavaScript or TypeScript developer.

pgorod commented 2 months ago

The labels and label colors are pulled directly from todoist, I'm guessing that if you're adding labels, without adding colors, that will have problems displaying.

(I'm just quickly looking at this on my mobile, haven't had a chance to test it)

But I wonder what would be the best way of displaying subtasks. Maybe we shouldn't be mixing that up with labels, which are a different concept. Maybe an indent would work? Or a smaller font?

I'm also curious as to how this works with sorting options, I don think it's easy to get the subtasks to always show under their respective tasks...

Thanks for your work, I'm looking forward to evolving this into a great new feature.

PavelD commented 2 months ago

Hi, I wrote in the 1st line it's POC or in other words MVP :-)

I just want to have some link between task and the subtask. I was considering to add prefix to the content filed (actually it looks like title of the task). But it brings new problems with the update of the subtask.

Label seems to me minimal change and sorting? Still better than before when subtasks were displayed together with common tasks.

pgorod commented 2 months ago

Can you post some screenshots of how it looks?

I'm currently reinstalling my OS and don't have a dev environmet set up

PavelD commented 2 months ago

Next week. I'm not close to my HA.