I'm supposing this is the code that writes dates on Github web front. <p dir="auto

Here’s a test case that illustrates the issue: <div class="highlight highlight-sou

<a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/us

Wrong human text about relative-time-element HOT 5 OPEN

antonio-fr commented on July 18, 2024 1

Wrong human text

from relative-time-element.

Comments (5)

keithamus commented on July 18, 2024 1

I have a refactor PR that’ll be ready soon which will resolve these without regressing the other cases we’ve run into.

from relative-time-element.

turbo commented on July 18, 2024

Same issue on https://github.com/grayjoking/backhand (flagged up by user):

from relative-time-element.

stapelberg commented on July 18, 2024

Here’s a test case that illustrates the issue:

--- a/test/relative-time.js
+++ b/test/relative-time.js
@@ -2411,18 +2411,18 @@ suite('relative-time', function () {
         expected: 'in 3 years',
       },
       {
-        reference: '2022-12-31T12:00:00.000Z',
-        datetime: '2024-03-01T12:00:00.000Z',
-        tense: 'future',
-        format: 'micro',
-        expected: '3y',
+        reference: '2023-02-18T19:00:00.000Z',
+        datetime: '2021-02-02T20:49:31.000Z',
+        tense: 'past',
+        format: 'auto',
+        expected: '2 years ago',
       },
       {
-        reference: '2021-04-24T12:00:00.000Z',
-        datetime: '2023-02-01T12:00:00.000Z',
-        tense: 'future',
-        format: 'micro',
-        expected: '2y',
+        reference: '2023-02-18T19:00:00.000Z',
+        datetime: '2021-01-23T09:28:06.000Z',
+        tense: 'past',
+        format: 'auto',
+        expected: '2 years ago',
       },
     ])

Output:

test/relative-time.js:

 ❌ relative-time > legacy formats > <relative-time datetime="2021-02-02T20:49:31.000Z" tense="past" format="auto"  > => 2 years ago
      AssertionError: expected '3 years ago' to equal '2 years ago'
      + expected - actual
      
      -3 years ago
      +2 years ago

Might be related to #249? cc @keithamus

from relative-time-element.

Wulfheart commented on July 18, 2024

@keithamus currently I am trying to dive into the code. Could you please point me to the PR with the refactoring?

from relative-time-element.

keithamus commented on July 18, 2024

I did some work on trying to create a polyfill for Temporal Durations so we could move to those but the work stalled due to the complexity of the Temporal APIs. I don't have an active branch or PR so the current code is the latest iteration.

from relative-time-element.

Recommend Projects