search

sinedied / hads

:books: Markdown superpowered documentation for Node.js
MIT License
167 stars 28 forks source link

Added last modified information on title header line #25

Closed fabala closed 6 years ago

sinedied commented 6 years ago

Thanks for the fix!

I was just wondering about the date format, as it's not i18n friendly (for example in US the day/month would be inversed). I saw that github does not use absolute date to avoid this issue but relative time like "25 minutes ago", I think that's smart and easier to track the time since changes were made.

Would that fit your use case?

If so, that could be implemented easily with momentjs: https://momentjs.com/docs/#/displaying/fromnow/ : moment(lastModificationDate).fromNow()

fabala commented 6 years ago

I had exactly the same question about the date format when I wrote this PR.

Indeed, this is a very good idea as to me. I gonna create a new PR with that, thanks!

sinedied commented 6 years ago

Perfect, thanks!