junlincc commented on pull request #11602: URL: https://github.com/apache/incubator-superset/pull/11602#issuecomment-723237857
> Another idea is to use dynamic font size based on text length. We already do that for [some visualizations](https://github.com/apache-superset/superset-ui/blob/master/plugins/legacy-preset-chart-big-number/src/BigNumber/BigNumber.tsx#L145-L151) and there is a utility function in `superset-ui` readily available to use. > > Personally I feel it makes more sense to wrap the title instead of truncating it because users put in long titles for a reason---if they really feel it looks bad, they would choose a shorter title already. Our original design/implementation is to have tooltip showed while user hover truncated title. so that user can view a cleaner dashboard without seeing overloaded information yet being able to access information as they need. My understanding is user have to put in long time is because 1. to include details for specific chart 2. there are no other easy places to store or display these detailed information We certainly shouldn't ask user to cut the necessary details associate with a chart, but we can certainly come up w better solution for the second point. Design input and user testing are needed for this topic. though this is a small change in the entire product, we try not to make product decision base on individual preference or a single use case. ---------------------------------------------------------------- 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]
