search

codex-team / task-manager

3 stars 2 forks source link

Added opportunity to edit task content #103

Closed slaveeks closed 2 years ago

slaveeks commented 2 years ago

resolves #80

talyguryn commented 2 years ago

no "saved" notifies. need to discuss it but i want to see any marker to be sure that content was saved. may be sign "saved" or any loader, i don't know yet

talyguryn commented 2 years ago

may i ask to change cursor to pointer when it moves to the dark area around the popup to click to close it?

talyguryn commented 2 years ago

is this warning ok?

image

talyguryn commented 2 years ago

when i update task content in popup and close it that card does not update. it is not good too

talyguryn commented 2 years ago

Added page reload, when popup closes

that is not the best solution. can we just update title in the tasks list and ask react to update the layout of that card automatically?

talyguryn commented 2 years ago

still see no marker for "saved success" state. is there any way to create it and implement?

need just to show a note "Saved" with opacity 50% somewhere for a 300ms

talyguryn commented 2 years ago

strange position of the right column below X, which should not affect popup content. it is good?

image

TatianaFomina commented 2 years ago

Added page reload, when popup closes

that is not the best solution. can we just update title in the tasks list and ask react to update the layout of that card automatically?

@slaveeks @talyguryn in https://github.com/codex-team/task-manager/pull/104 added updating tasks data in list on status change (store is used for that purpose). Maybe we should consider merging mentioned pr first and then use the same approach here