search

BlitzKraft / saythanks.io

Spreading Thankfulness in Open Source.
https://saythanks.io
1.25k stars 154 forks source link

Timestamp each note in the inbox #157

Closed kgashok closed 2 years ago

kgashok commented 2 years ago

It would be good if the time of when the note was sent is also displayed. Which means, the note has to be timestamped with the date (as per the time zone).

williskneeland commented 2 years ago

I'd like to try adding this!

kgashok commented 2 years ago

@williskneeland , all yours.

Deepthi-Deepu commented 2 years ago

I would like to work on this

williskneeland commented 2 years ago

Feel free to give this to someone else. I'm too busy currently to work on it, apologies.

kgashok commented 2 years ago
  1. The title doesn't have to be "timestamp" - it can say "Date", or any other label - what is used in other email systems?
  2. The Date does not have to be bold in font style
  3. The size of the font can be smaller so it more or less fits into one line
  4. If the date is recent, we can limit it to month and day, and only if it is a very old date, then include the year.

image

kgashok commented 2 years ago

@Deepthi-Deepu - incrementally fix each of the above issue. Don't fix everything in one shot and make a PR. Thanks,

kgashok commented 2 years ago

and with Verdana, it is three lines:

image

kgashok commented 2 years ago

image

Deepthi-Deepu commented 2 years ago

I will make changes and update it soon

kgashok commented 2 years ago

Closing this issue for now, @Deepthi-Deepu @Pavithratrdev

Super work!