kasiazjc commented on PR #36934: URL: https://github.com/apache/superset/pull/36934#issuecomment-3720379773
@luizotavio32 Hi, is this change configurable somewhere? Like is it chart/dashboard setting? From a UX standpoint, this feels like a very specific and potentially noisy piece of information, and not something all users will want to see by default. Because of that, users should have control over its visibility/being able to turn it off or on A few concerns and suggestions: - Default behavior: This should be turned off by default to avoid introducing unexpected visual changes to existing dashboards. Preserving current layouts and, in general visual consistency is important, esp for production dashboards - Dashboard context: In most dashboard use cases, users expect all charts to refresh at load time, meaning this timestamp would usually be identical across charts. Showing it everywhere adds little informational value while increasing visual clutter - _I think_ currently user can set auto refresh only for the whole dashboards and not specific charts, so again, needing it in every chart may be useful only in specific use cases Some thoughts on configurability: - At the dashboard level, a global setting could make sense for teams that find this information useful (turned off by default) - At the chart level, I think it is ok to display it by default, but the timestamp should appear at the bottom of the chart - like it is displayed in dashboard view, and to avoid competing with primary chart content Overall, I think this feature makes sense for some users, but only if it’s optional and can be configured, I think, per dashboard cc: @michael-s-molina @yousoph -- 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]
