RGJ
12 Dicembre 2022, 4:33pm
1
Mi sono appena imbattuto in questo. Guarda l’ora nel popup di sinistra quando clicco primo-secondo-primo.
Ad esempio, 2022-12-13T14:00:00Z sarebbe l’ideale, anche se probabilmente potrei anche usare 2022-12-13T13:30:00Z ?
Succede solo quando le due date sono nello stesso paragrafo, quindi premere invio prima di “anche se” risolve il problema.
7 Mi Piace
Ho giocato con la funzione calendario e ho trovato delle cose strane, potrebbe essere correlato al bug.
2 Mi Piace
ted
15 Dicembre 2022, 2:58am
5
Ottima osservazione! :occhio:
Quando abbiamo aggiunto il supporto dedicato per gli intervalli di date, abbiamo anche aggiunto un fallback per il vecchio modo “automatico” di rilevarli (ad esempio, ci sono due date nel paragrafo, quindi consideralo un intervallo).
Questo fallback si basa sul controllo del numero di elementi figlio nel paragrafo. Quando si espande la modale, anche l’elemento modale viene aggiunto come elemento figlio. Questo è il motivo per cui ci sono risultati diversi quando si fa clic su una data, a seconda che una modale sia già aperta o meno.
Sto lavorando a una correzione.
4 Mi Piace
ted
12 Gennaio 2023, 10:58am
6
Questo è stato risolto in PR#19477 .
main ← Drenmi:fix/remove-magic-time-ranges
opened 06:30AM - 15 Dec 22 UTC
### History
In #15474 we introduced dedicated support for date ranges. As par… t of that change we added a fallback of "magic" date ranges, which treats dates in any paragraph with _exactly_ two dates as a range. There were discussions about migrating all such paragraphs to use the new date range element, but it was ultimately decided against.
### What's the problem?
1️⃣ A minor [bug](https://meta.discourse.org/t/timezone-bug/248674) was reported relating to the "magic" date range. If a Tippy was already open when expanding a date, the date would be considered a single date, but if no Tippy was open, it would be considered a range. So expanding the same date could have different results based on the initial conditions.
(The reason this happened is the fallback relies on the number of elements in the paragraph being exactly 2, but the Tippy element was also added as a child of the same paragraph, resulting in 3 children.)
2️⃣ While working on this, I additionally found that using two date ranges in the same paragraph would result in all dates in the paragraph being treated as single dates, rather than being grouped together in ranges.
### How does this fix it?
Firstly, we remove the fallback to "magic" date ranges, as decided. This fixes the 1️⃣ problem.
Secondly, when looking at an element and it's parent paragraph, we now select all date range children and partition them two-by-two. This fixes the 2️⃣ problem.
### In summary
- Single dates are now always rendered as single dates.
- Multiple date ranges in the same paragraph now works correctly.
### In action
5 Mi Piace
