villebro commented on pull request #12008: URL: https://github.com/apache/incubator-superset/pull/12008#issuecomment-745573343
This looks awesome! Some comments (will review code separately): 1. I'm not able to replicate @junlincc's error messages reported above. It looks like there's something wrong with the perms and needs `superset init` to be run. 2. I can also confirm the missing scroll on columns and metrics. 3. The dataset name probably doesn't need to show the name on hover (see video below). 4. The dataset options appear even without a click, is this intentional? (see video below where no mouse clicks happen).  ---------------------------------------------------------------- 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]
