ktmud opened a new issue #13472:
URL: https://github.com/apache/superset/issues/13472


   **Is your feature request related to a problem? Please describe.**
   
   https://github.com/apache/superset/pull/13057 added "timeseries limit" and 
"sort by" to various multi-series charts. But it's not clear the "Sort by" 
field applies to only the series limit, not the total rows.
   
   <img 
src="https://user-images.githubusercontent.com/335541/110073332-10e2c300-7d34-11eb-9bdc-dc13861ba375.png";
 width="400">
   
   **Describe the solution you'd like**
   
   We should hide the "Sort by" control using chart control's 
[`visibility()`](https://github.com/apache-superset/superset-ui/blob/4137011e0ef6ef2ec774e03027fcf894621fb1d7/packages/superset-ui-chart-controls/src/types.ts#L176)
 API unless "Series limit" is set and not `0`.
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to