Skip to content
This repository has been archived by the owner on Feb 21, 2025. It is now read-only.

"Matches resume at" should say "tomorrow" #11

Open
PeterJCLaw opened this issue Jul 11, 2020 · 3 comments
Open

"Matches resume at" should say "tomorrow" #11

PeterJCLaw opened this issue Jul 11, 2020 · 3 comments
Assignees

Comments

@PeterJCLaw
Copy link

PeterJCLaw commented Jul 11, 2020

Rather than just a time, for cases when the timings are the next day.

@trickeydan
Copy link
Contributor

This should have been handled by moment js :/

@trickeydan trickeydan self-assigned this Jul 12, 2020
@trickeydan
Copy link
Contributor

let startTime=new Date(this.target);
            hours = startTime.getHours();
            minutes = startTime.getMinutes();

            hours = hours < 10 ? "0" + hours : hours;
            minutes = minutes < 10 ? "0" + minutes : minutes;

            $("#match-counter .timer .clock").text(hours + ":" + minutes)

Looks like the code doesn't handle the date at all.

@RealOrangeOne
Copy link
Member

The functionality of showing raw hours does reflect how the competition screens display it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants