[feature proposal] Local time zone in timestamps

[oscarlofwenhamn]

• Gitea version (or commit ref): 1.7.1
• Git version: n/a
• Operating system: RHEL 7.5
• Database (use [x]):
  • PostgreSQL
  • MySQL
  • MSSQL
  • SQLite
• Can you reproduce the bug at https://try.gitea.io:
  • Yes (provide example URL)
  • No
  • Not relevant
• Log gist: n/a

Description

As a user, I would like to (at least have the option to) see timestamps presented in my local timezone. As several users can be located in different time zones, it would be a better experience to see timestamps presented in a way that corresponds to the local time.

My local solution to this is a short script that adjusts the stamp using the js Date-function placed in the footer:

window.onload=function(){
    var times = document.getElementsByClassName("time-since");
    for (var i = 0; i<times.length; i++){
		var localTime = new Date(times[i].getAttribute("data-content"));
		if(localTime != "Invalid Date"){
			times[i].setAttribute("data-content", localTime.toString());
		}
    }
}

though as I am not very experienced in javascript, a better solution is probably possible. Regardless of implementation, this is my proposal.

Screenshots

Current

Proposed

[oscarlofwenhamn]

In order to get rid of the "(Central European Standard Time)", i added a regex replace, ending up with

localTime.toString().replace(/ *\([^)]*\) */g, "")

which brings it all down to a more similar ui size as the original.

[NateScarlet]

Good work
Maybe you should use localTime.toLocaleString
This function give a translated result

[NateScarlet]

A userscript to use with current version:

// ==UserScript==
// @name     Gitea local time
// @version  1
// @run-at   document-end
// @include  *
// ==/UserScript==

(function() {
  /**
   * @type {HTMLMetaElement}
   */
  const meta = document.querySelector('meta[name=keywords]');
  if (!(meta && meta.content.split(',').includes('gitea'))) {
    return;
  }
  /**
   * @type {NodeListOf<HTMLSpanElement>}
   */
  const times = document.querySelectorAll('span.time-since.poping.up');
  for (const i of times) {
    const localTime = new Date(i.dataset.content);
    if (isNaN(localTime)) {
      continue;
    }
    i.dataset.content = localTime.toLocaleString();
  }
})();

[lunny]

@NateScarlet could you send a PR to fix that?

[oscarlofwenhamn]

Brilliant, nice addition @NateScarlet. And thanks for the help with a PR, I haven't been sure enough of my solution and the workflow to make one.