Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Explore] Time picker custom setting for timezone #12302

Closed
graceguo-supercat opened this issue Jan 6, 2021 · 10 comments
Closed

[Explore] Time picker custom setting for timezone #12302

graceguo-supercat opened this issue Jan 6, 2021 · 10 comments
Assignees
Labels
enhancement:committed Enhancement planned / committed by a contributor explore:time Related to the time filters in Explore inactive Inactive for >= 30 days

Comments

@graceguo-supercat
Copy link

graceguo-supercat commented Jan 6, 2021

  • In Advanced range type, user can type in time with free text (this is great! l like this design and implementation!!):

advanced

  • But in Custom range type, user have to set time by click on hour/minute/second, free text will be overwritten when user close calendar.
    b8psB5uD5r

  • I think the 3 columns for hour/minute/second is a little extra.

  • The settings for Advanced vs Custom should be consistent: one place allow user type free text, the other is not, with no obvious reason or hint from UI. When i type in time in Custom range type and then get removed , i thought it was a bug.

Environment

latest mater

@junlincc @zhaoyongjie

@graceguo-supercat graceguo-supercat added P1 Priority item - Major explore:time Related to the time filters in Explore #bug Bug report labels Jan 6, 2021
@graceguo-supercat graceguo-supercat changed the title [Explore] Timepicker Advanced/Custom time setting is not consistent [Explore] Timepicker Advanced/Custom range type: time setting is not consistent Jan 6, 2021
@graceguo-supercat graceguo-supercat changed the title [Explore] Timepicker Advanced/Custom range type: time setting is not consistent [Explore] Timepicker Advanced/Custom range type: time settings is not consistent Jan 6, 2021
@junlincc junlincc added enhancement:committed Enhancement planned / committed by a contributor and removed #bug Bug report labels Jan 6, 2021
@junlincc
Copy link
Member

junlincc commented Jan 6, 2021

Thanks for the nice comment for Advanced! @graceguo-supercat
much more users can do in the free text input, and it reaches full feature parity with Tableau!
if you think below would be useful for your users, pls also help test them and give us feedback 🙏 @zuzana-vej @etr2460
Screen Shot 2021-01-06 at 6 44 36 AM

@junlincc
Copy link
Member

junlincc commented Jan 6, 2021

again, thanks for the valuable feedback, we will get this enhancement done by 1.0 release.
The settings for Advanced vs Custom should be consistent

@zhaoyongjie 🙏

@graceguo-supercat
Copy link
Author

This specifications for the new Timepicker is great. I will share with airbnb users after our next release.

@zhaoyongjie
Copy link
Member

zhaoyongjie commented Jan 7, 2021

Hi, @graceguo-supercat. There are some UX usage issues here.

  • In the Custom Frame:
    The new Datepicker(Antd Datepicker) use mouse onBlur event as shortcut to cancel value input. (so the Datepicker value cannot be saved in time)
    If you need to confirm the change, you need to use the enter key or mouse click the Ok button. (submitted Datepicker value)
Jan-07-2021.21-21-19.mp4
  • For Advanced Frame vs Custom Frame consistent
    Custom Frame is just a subset of the Advanced Frame representation range, Advanced Frame has more rich datetime-expression. Datapicker in the Custom Frame only accepts YYYY-MM-DD[T]HH:mm:ss datetime style, while Advanced Frame is a free-text input box.

CC: @junlincc

@graceguo-supercat
Copy link
Author

graceguo-supercat commented Jan 8, 2021

Thanks @zhaoyongjie I didn't realize enter key is the key :)

I have a couple of other requests:

  • Midnight in Custom type:

is it Midnight UTC time zone? or user's local time zone? Is it the last midnight or the next midnight in the future? Please make it clear:
Screen Shot 2021-01-07 at 3 56 48 PM

  • Available date operations in Advanced type:

Screen Shot 2021-01-07 at 4 11 50 PM

Advanced date type has many new operators, like `dateadd`, `datetrunc` etc, which we don't have before. We should offer a full list of feature descriptions for users. Otherwise for those non-tech background users, they don't know how to use it or how to find the secret.

So i am thinking better way to expose advanced operations, for example:

  • add a big tooltip for Config Advanced Time Range, and list many usage samples like @junlincc shared, or
  • something like Markdown component, it links out to a Markdown Cheatsheet.

cc @junlincc please give some suggestions?

@graceguo-supercat graceguo-supercat removed the P1 Priority item - Major label Jan 8, 2021
@junlincc
Copy link
Member

junlincc commented Jan 8, 2021

is it Midnight UTC time zone?

Yes, it's UTC time zone, and we have plan to allow user to set time zone. it's a committed enhancement in roadmap @graceguo-supercat

@junlincc
Copy link
Member

junlincc commented Jan 8, 2021

Advanced date type has many new operators, like dateadd, datetrunc etc, which we don't have before. We should offer a full list of feature descriptions for users. Otherwise for those non-tech background users, they don't know how to use it or how to find the secret.

Yes! I am well aware of the need of adding tooltip of advanced syntax description, as I'm one of the non-tech background users😅. We will get both projects started within a couple weeks. @graceguo-supercat

@junlincc junlincc changed the title [Explore] Timepicker Advanced/Custom range type: time settings is not consistent [Explore] Time picker custom setting for timezone and adding tooltip for Advanced Jan 8, 2021
@junlincc junlincc changed the title [Explore] Time picker custom setting for timezone and adding tooltip for Advanced [Explore] Time picker custom setting for timezone Feb 16, 2021
@stale
Copy link

stale bot commented May 2, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. For admin, please label this issue .pinned to prevent stale bot from closing the issue.

@stale stale bot added the inactive Inactive for >= 30 days label May 2, 2022
@rusackas
Copy link
Member

Closing this as stale... the time picker has undergone a lot of redesign and reengineering since this was opened, and will continue to do so in the future. I think the most actionable parts have been handled, and any remaining bugs and issues should be handled in new/updated Issues and Discussions as is most fitting.

@ThanhTrung90
Copy link

is it Midnight UTC time zone?

Yes, it's UTC time zone, and we have plan to allow user to set time zone. it's a committed enhancement in roadmap @graceguo-supercat

@junlincc Can user set time zone in superset right now?
If yes, how can user do that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement:committed Enhancement planned / committed by a contributor explore:time Related to the time filters in Explore inactive Inactive for >= 30 days
Projects
None yet
Development

No branches or pull requests

5 participants