kasiazjc commented on PR #36934: URL: https://github.com/apache/superset/pull/36934#issuecomment-3723140475
> > > 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 > > > > > > > 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 > > > > > > Yeah, a dashboard configuration makes sense (off by default). This is a long standing request from our users and an option offered by other BI tools. I do agree though that it might be useful depending on the dashboard type and the frequency of updates. > > Agreed. > > @kasiazjc @michael-s-molina where should we add the on/off config? We have some options like the dashboard properties modal or the left sidebar that appears when clicking `Edit dashboard` button Thanks @michael-s-molina for feedback 🙏 @luizotavio32 My first thought was to actually add it to dashboard properties, but I think it makes much more sense to add it in the dashboard "more" dropdown, since this is already where users manage auto-refresh, force refresh etc, so it's kept in context. I think it could look like this: <img width="301" height="357" alt="image" src="https://github.com/user-attachments/assets/34e20e76-c54e-433f-b5a8-2f9f904e9945"; /> This _could_ also have a tooltip, but if you think the label is enough I am good with that. If we decide on including tooltip though, we could do something like Displays the `“Last queried” timestamp below each chart on the dashboard` In addition, I’ve been thinking about a couple of lightweight enhancements that build on existing patterns: - In the chart dropdown on dashboards, we already show “Force refresh” and “Cached X time ago”. We could add a tooltip on hover over the cached text to show the exact query time. <img width="536" height="484" alt="image" src="https://github.com/user-attachments/assets/a9d4b0d6-7411-4036-beb5-061c49e68616"; /> - In dashboard “More” settings, where we have “Refresh dashboard” (essentially the same as force refresh), we could also display “Cached X time ago” underneath, the same as in the chart dropdown. On hover, this could reveal more detail for example, when the dashboard was fully loaded - if it does not make sense to show - let me know! Wdyt @luizotavio32 @michael-s-molina ? -- 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]
