You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/01/26 14:52:18 UTC

[GitHub] [airflow] ryanahamilton commented on pull request #13904: Only show User's local timezone if it's not UTC

ryanahamilton commented on pull request #13904:
URL: https://github.com/apache/airflow/pull/13904#issuecomment-767595814


   @jedcunningham I like this update. I'm wondering if it might be better to make the "UTC" timestamps themselves conditional instead (when "Local" is not UTC)? Very similar result (see below), but then you'll consistently have a labeling/confirmation of what the "Local" timezone is. If you agree with that strategy, I'd also suggest swapping the order so the "Local" timestamps come before the "UTC" timestamps as that will keep all of the tooltips more consistent with the conditional info at the tail.
   
   ![image](https://user-images.githubusercontent.com/3267/105860283-66041a00-5fbb-11eb-9972-e23946bff218.png)
   


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