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

Contact History: Show time of an event #1809

Closed
AkhileshNegi opened this issue Dec 30, 2021 · 4 comments · Fixed by #1838
Closed

Contact History: Show time of an event #1809

AkhileshNegi opened this issue Dec 30, 2021 · 4 comments · Fixed by #1838
Assignees
Labels
enhancement New feature or request

Comments

@AkhileshNegi
Copy link
Member

Is your feature request related to a problem? Please describe.
In contact history, we show only the date of a contact history event and not the time, though the events are sorted in descending order, having time corresponding to each entry would be much informative

Describe the solution you'd like
Show the timing of the contact history event in-between event and date

Screenshot
image

@AkhileshNegi AkhileshNegi added the enhancement New feature or request label Dec 30, 2021
@abhi1203
Copy link

abhi1203 commented Jan 4, 2022

@AkhileshNegi
here: https://www.sketch.com/s/ba323b85-11d7-4687-8a19-681a32132bab/a/81A5a7
we'll need to increase 1 column space for the input fields above also

@kurund kurund self-assigned this Jan 7, 2022
@kurund kurund linked a pull request Jan 9, 2022 that will close this issue
@kurund
Copy link
Contributor

kurund commented Jan 9, 2022

@abhi1203

This is the only place we are using am/pm. Also showing 9/01/2022 instead of 09/01/2022.

Let me know your thoughts?

@abhi1203
Copy link

Hey @kurund im okay with 24hr clock also if we've used that. 12 hrs might be confusing without am/pm. and i needed more space so i shortened the date fo 9/01/22 we can use full 9/01/2022 alsoo since widening this section will give some space

@kurund
Copy link
Contributor

kurund commented Jan 10, 2022

@abhi1203

Thank you. I have updated the date format to be consistent with other date-time interfaces.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants