Theorem patch with cleveref and amsthm loaded
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 44/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- latex
- Área
- tooling
Línea de trabajo
Comience con el parche de refstepcounter en hyperref.dtx, en las líneas 11129–11135, y siga después cómo cleveref, junto con amsthm, redefine @thm mediante cref@thmnoarg y cref@thmoptarg. Determine cómo deberían recibir el parche del teorema ambas rutas, incluido el caso con argumento opcional, y verifique el comportamiento de esta combinación de paquetes con respecto a los problemas señalados en los issues #336 y latex2e #1779.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The theorem patch does the following:
https://github.com/latex3/hyperref/blob/d2eb2fae09eee648f81659613a37e3e45566e479/hyperref.dtx#L11129-L11135
If cleveref is loaded, it hooks some spacing stuff in there. I guess it's because cleveref with amsthm loaded redefines things with
\let\cref@thmnoarg\@thm%
\def\@thm{\@ifnextchar[{\cref@thmoptarg}{\cref@thmnoarg}}%]
Or maybe there's another reason I don't understand.
Anyway, what I'm confused about is the \refstepcounter patch in \@thm. If cleveref is loaded, then this patch isn't being done at all, but why not?
Patching \@thm no longer works, but patching \cref@thmnoarg seems like it does.
For completeness, \cref@thmoptarg should probably also be patched then, but that would probably also require a variant of \Hy@theorem@refstepcounter with an optional argument, like \cref@thmoptarg wants to use it.
I guess this might be related to #336 and https://github.com/latex3/latex2e/issues/1779. And perhaps amsthm should just be improved like https://github.com/latex3/latex2e/pull/1301, instead of piling more patches on top of each other for this package combination.
- Lenguaje dominante
- TeX
- Estrellas
- 206
- Forks
- 41
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de latex3/hyperref
-
Dificultad 3/5 1-2 días Aptitud para principiantes 58/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 58/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Todos los issues de latex3/hyperref
Issues similares
-
enhancement needs triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
elastic/docs-builder#4495 ·
Los mantenedores suelen responder en 1 día
-
[Bug]: Target width input clamps the value on every keystrokePosiblemente ocupada @AnushreeKasturi la tomó hoy. Abiertobug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
suradet-ps/codeframe#37 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 2 días
-
change request evaluating
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
sindresorhus/eslint-plugin-unicorn#3907 ·
Los mantenedores suelen responder en 1 día
-
bug needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día