sadpandajoe commented on code in PR #44127: URL: https://github.com/apache/superset/pull/44127#discussion_r4011053655
########## docs/docs/using-superset/exploring-data.mdx: ########## @@ -165,6 +165,19 @@ control can override a simple metric's own aggregation (to Sum or Average) for t only — metrics built from custom SQL keep their own aggregation regardless. ::: +In the **Options** panel, the **Show values as** control displays each cell as a percentage of its +row total, column total, or grand total instead of its actual value: + +:::note +Totals and subtotals are still computed correctly first, as described above; **Show values as** +only changes how the already-correct cell, row-total, column-total, and grand-total values are +displayed, not how they're calculated. Switching it does re-run the query, though: for non-additive Review Comment: This control has no `renderTrigger`, so selecting another value only marks the chart stale; it does not itself issue the requery that this sentence promises. Could this say that changing the option *requires* rerunning the query so users know to click **Run Query**? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
