RGJ
12 Diciembre, 2022 16:33
1
Acabo de encontrar esto. Mira la hora en el popup de la izquierda cuando hago clic en primero-segundo-primero.
Por ejemplo, 2022-12-13T14:00:00Z sería ideal, aunque probablemente también podría usar 2022-12-13T13:30:00Z ?
Solo ocurre cuando las dos fechas están en el mismo párrafo, por lo que presionar enter antes de “aunque” resuelve el problema.
7 Me gusta
Jugué con la función del calendario y encontré algunas cosas extrañas, podría estar relacionado con el error.
2 Me gusta
ted
15 Diciembre, 2022 02:58
5
¡Gran observación!
Cuando agregamos soporte dedicado para rangos de fechas, también agregamos una solución alternativa para la antigua forma “automágica” de detectarlos (es decir, ¿hay dos fechas en el párrafo?, si es así, considéralo un rango).
Esta solución alternativa se basa en verificar el número de elementos secundarios en el párrafo. Al expandir el modal, el elemento modal también se agrega como un elemento secundario. Es por eso que hay diferentes resultados al hacer clic en una fecha, dependiendo de si un modal ya está abierto o no.
Trabajando en una solución.
4 Me gusta
ted
12 Enero, 2023 10:58
6
Esto se ha corregido en 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 Me gusta
