reactjs / reactjs/react.dev

Clock examples use impure `.toLocaleTimeString()`

Abierto
#5,136 0 comentarios 1 reacción 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
JavaScript
Estrellas
11.8k
Forks
7.9k
Merge medio
1 d 11 h
PR fusionados (30 d)
11

Descripción

The new React Docs have various "Clock" examples and challenges throughout the page. All of them use .toLocaleTimeString() (without arguments!) to output the time. This is a bit strange, especially as an example on the page "Keeping Components Pure":

While .getHours() will always return the same value if the Date object is the same, the whole point of .toLocaleTimeString() with no locale specified is that it's impure, its return value depending on external state outside React: the configured environment locale.

If you scroll just a little bit upwards on the page "Keeping Components Pure", you'll find

Your components could run in a different environment—for example, on the server!

as one of the reasons for why React cares about component purity. But that example is very likely to break with .toLocaleTimeString(), as chances are that the server has a different default locale than the user's browser.

My suggestion would be to pass a language prop to the Clock component, so Clock itself is actually pure, but App (which isn't a visible part of the example) can do <Clock time={time} language={navigator.language} />, keeping the nice behavior of localized examples: https://codesandbox.io/s/sandpack-project-forked-43udl3?file=/Clock.js

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Busca en los nuevos React Docs los ejemplos de Clock y revisa el desafío “Keeping Components Pure”, junto con el ejemplo vinculado Clock.js. Actualiza los ejemplos para que Clock reciba el contexto de idioma mientras se conserva la salida localizada y, después, verifica que todos los ejemplos de Clock de la página sigan el mismo patrón.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
javascript, react
Área
documentation, frontend
Tipo de issue
Documentación
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
48/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.