[GitHub] [superset] craig-rueda commented on pull request #24449: feat: custom refresh frequency

["craig-rueda (via GitHub)" <gi...@apache.org>]

craig-rueda commented on PR #24449:
URL: https://github.com/apache/superset/pull/24449#issuecomment-1599189033

   From the screenshots, it looks a little confusing to show the dropdown alongside the text fields. Perhaps a new "virtual" option (`CUSTOM`) should be added, which would then hide/show the text boxes when selected? Also, having two buttons when choosing a custom refresh interval looks a little confusing as well. 


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org