[GitHub] [incubator-pinot] npawar commented on pull request #6327: Added proper tooltips, ability to enable-disable table state and fixed page crash on reload status

[GitBox <gi...@apache.org>]

npawar commented on pull request #6327:
URL: https://github.com/apache/incubator-pinot/pull/6327#issuecomment-740125975


   Thanks @shahsank3t ! Few missing tooltips and comments:
   
   1. Add tooltip for Estimated size to individual table page
   2. Add tooltip for Reported size, estimated size, status to the Tables page
   3. Transform functions link broken on Add Offline page
   4. Link for Indexing and Encoding missing
   5. For some links (noticed in Filter and transform function, and all links on REALTIME table page as well), it is opening in same page instead of new tab. Can we make it always open in new tab?
   6. Relocate to tag tooltip missing
   7. StreamConfigs tooltip appears far away and disappears when trying to click
   8. Repeat all tooltips from the OFFLINE table to the REALTIME table form
   9. The “i”  in the purple circle UI for the links was not very intuitive for me. I kept expecting some info to pop up. Any other way to show it? The “click here for details” you’ve added to some tooltips is great.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org