[IMP] spreadsheet: expanding pivot table documentation#17973
Conversation
|
|
xpl-odoo
left a comment
xpl-odoo
left a comment
There was a problem hiding this comment.
Great job @mcsi-odoo! I like the new structure and how the various concepts are explained. You managed to tackle yet another complex topic thoroughly, thanks a lot for this :)
There was a problem hiding this comment.
Is it worth it to have a separate page just for this? Or could this be moved elsewhere (or removed)?
There was a problem hiding this comment.
I've flip-flopped on this a lot! I don't want to just squeeze it in somewhere else, especially as I do think it's valuable for users who might be using Spreadsheet / Dashboard for financial reports and dashboards. Audrey was ok with me keeping it, especially with the idea that breaking up content into smaller chunks is in itself not a bad thing. So, unless you feel very strongly, I'd like to keep it (for now, at least).
xpl-odoo
left a comment
xpl-odoo
left a comment
There was a problem hiding this comment.
Great job @mcsi-odoo! I like the new structure and how the various concepts are explained. You managed to tackle yet another complex topic thoroughly, thanks a lot for this :)
There was a problem hiding this comment.
I've flip-flopped on this a lot! I don't want to just squeeze it in somewhere else, especially as I do think it's valuable for users who might be using Spreadsheet / Dashboard for financial reports and dashboards. Audrey was ok with me keeping it, especially with the idea that breaking up content into smaller chunks is in itself not a bad thing. So, unless you feel very strongly, I'd like to keep it (for now, at least).
| To :ref:`prevent real-time updates <spreadsheet/pivot-tables/properties-defer-updates>` while | ||
| building or manipulating a pivot table, enable :guilabel:`Defer updates`. |
There was a problem hiding this comment.
I prefer to keep it here since this is an overview of what you see in the properties panel, so it would feel odd to not mention it.
| To change how a measure is shown: | ||
|
|
||
| #. Open the :ref:`pivot table's properties panel <spreadsheet/pivot-tables/properties>`. | ||
| #. On the measure's card, click the :icon:`fa-cog` :guilabel:`Show values as` icon. |
There was a problem hiding this comment.
Instead, I've updated this to:
| #. On the measure's card, click the :icon:`fa-cog` :guilabel:`Show values as` icon. | |
| #. On the measure's card, click :icon:`fa-cog` :guilabel:`(Show values as)`. |
Since there is actually a label when you hover over it. Looks ok?
mcsi-odoo
force-pushed
the
19.0-spreadsheet-pivot-tables-mcsi
branch
from
0adb93d to
cbe6c20
Compare
|
Thanks for addressing my suggestions and for providing some context if not ;) @robodoo r+ |
task-6048514 closes #17973 Signed-off-by: Xavier Platteau (xpl) <xpl@odoo.com>
task-6048514 closes #17973 Signed-off-by: Xavier Platteau (xpl) <xpl@odoo.com>
3 participants
task-6048514