[FIX] account: uneditable readonly section / note lines #200155
Conversation
|
|
svfu-odoo
requested review from
a team and
aboo-odoo
and removed request for
a team
There was a problem hiding this comment.
Hello 👋
I'm okay with the readonly part but I believe we should keep the placeholder. Maybe modify it because it's true that it does not reflect the user's possibilities when the move is posted but removing it causes another issue.
That issue is that the readonly part of the template is used whenever focus is lost on the line meaning also unfocused line in a draft entry. Because of that, section or note lines without text on a draft entry used to look like this:
and look like this now:
Replacing the placeholder by simply "Section" and "Note" feels more appropriate imo. So that it fits both when the user is able of modify it and when he is not. What do you think ?
Currently one can still edit section / note lines on posted moves (but not save the move). This commit makes the section / notes lines readonly when they should be. task: none
svfu-odoo
force-pushed
the
saas-17.4-readonly_note_section_lines-svfu
branch
from
7fd2904 to
e224996
Compare
|
Good catch, thanks |
Currently one can still edit section / note lines on posted moves (but not save the move). This commit makes the section / notes lines readonly when they should be. closes #200155 Task: none Signed-off-by: Olivier Colson (oco) <oco@odoo.com>
5 participants
Currently one can still edit section / note lines on posted moves (but not save the move).
This commit makes the section / notes lines readonly when they should be.
task: none