[GitHub] [superset] etr2460 edited a comment on pull request #17751: fix: [alert] add tooltip message for alert threshold value

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

etr2460 edited a comment on pull request #17751:
URL: https://github.com/apache/superset/pull/17751#issuecomment-994087856


   I guess why let the user type in something invalid at all? We're adding the tooltip, but that would only be seen if you hover over it. You wouldn't let a user type letters into a credit card number input (for example), so why not filter out/disallow decimal points from the text here?


-- 
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