You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2020/12/11 00:07:16 UTC
[GitHub] [incubator-superset] mihir174 commented on pull request #11418: feat: Explore time picker enhancement
mihir174 commented on pull request #11418:
URL: https://github.com/apache/incubator-superset/pull/11418#issuecomment-742879353
Hey @zhaoyongjie, thanks for this! Looks great. Here's some design input
Flow for the advanced time range (reflects button states and error messages) -
https://www.figma.com/file/JWaGztdhZS0kS5ruG7x9tB/Control-Panel?node-id=491%3A46097
Button states-
- validate is only active when an unvalidated change has been made
- apply is only active when changes have been validated and the time range is valid
Error Message -
Invalid Time Range: Start date cannot be after end date
I've attached the error icon as an svg zip
The buttons should also use the Superset styles in storybook
![Edit Time Range Modal - Advanced](https://user-images.githubusercontent.com/64227069/101844563-8eb27c00-3b01-11eb-9f6f-b04d08133799.png)
[error_solid.svg.zip](https://github.com/apache/incubator-superset/files/5676320/error_solid.svg.zip)
----------------------------------------------------------------
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: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org