[IMP] studio: updating datetime field info #12350
Conversation
|
|
mcsi-odoo
force-pushed
the
17.0-studio-datetime-fields-update-mcsi
branch
2 times, most recently
from
372184b to
243bb48
Compare
There was a problem hiding this comment.
Hi @mcsi-odoo thank you for your work; this is definitely worth documenting!
I think the screenshot you added is a great idea and really helps clarify the different options. One general note about screenshots (and I know you didn’t do this): would you consider removing the center alignment for all of them 🙈? (You don't have to 😉 )
| - :guilabel:`Widget`: To change the default appearance or functionality of a field, select one of | ||
| the available widgets. | ||
| - :guilabel:`Placeholder`: To provide an example of how a field should be completed, write it under | ||
| :guilabel:`Placeholder`. It appears in light gray as a placeholder until a value is entered. |
There was a problem hiding this comment.
I know you didn't change this, but I don't think repeating the field's label is necessary?
mcsi-odoo
force-pushed
the
17.0-studio-datetime-fields-update-mcsi
branch
2 times, most recently
from
0d5ebf5 to
f0e2651
Compare
There was a problem hiding this comment.
Thank you for the changes @mcsi-odoo! I only have a few minor comments 🙂
This did, however, open another can of worms for me 🙈 but I think it's best to address it in a separate task and PR, especially since it's not urgent at all.
(Also see my comment on line 201) I actually think we could restructure/review the page a bit to:
- Explain how to add a field and mention the general properties at the top of the page (they are a bit lost at the bottom right now).
- Move the Date & Time properties to the relevant section (before the widget-specific sections).
- Look into documenting other field/widget-specific properties for other field types as well.
What do you think?
Task-4366107
mcsi-odoo
force-pushed
the
17.0-studio-datetime-fields-update-mcsi
branch
from
f0e2651 to
850bb1f
Compare
Task-4366107 closes #12350 Signed-off-by: Audrey Vandromme (auva) <[email protected]>
Task-4366107