You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Both hyperref and thmtools use kvsetkeys as key-val setting mechanism. Unfortunately thmtools globally patches kvsetkeys to prevent expansion of key name, which causes compatibility issue with how hyperref handlers unknown key passed to pdfinfo. See a detailed analysis in muzimuzhi/thmtools#34 (comment).
I fully understand that it's all thmtools' fault to patch hence change the behavior of kvsetkeys. But given currently it's hard to know why such patch was needed at that time (at least 10 years ago) and whether removing or modifying it will cause regression problems, could hyperref adapt to not use macro as key passed to \kv@parse@normalized?
sure no problem. In the long run hyperref will stop to use kvoptions anyway. Regarding theorems: I'm testing in the testthm branch if one can simplify the theorem patches and correct the problem that destinations are lost if a theorem starts with a list. Could you test if this gives (obvious) problems with thmtools?
Regarding theorems: I'm testing in the testthm branch if one can simplify the theorem patches and correct the problem that destinations are lost if a theorem starts with a list. Could you test if this gives (obvious) problems with thmtools?
Both
hyperref
andthmtools
usekvsetkeys
as key-val setting mechanism. Unfortunatelythmtools
globally patcheskvsetkeys
to prevent expansion of key name, which causes compatibility issue with howhyperref
handlers unknown key passed topdfinfo
. See a detailed analysis in muzimuzhi/thmtools#34 (comment).I fully understand that it's all
thmtools
' fault to patch hence change the behavior ofkvsetkeys
. But given currently it's hard to know why such patch was needed at that time (at least 10 years ago) and whether removing or modifying it will cause regression problems, couldhyperref
adapt to not use macro as key passed to\kv@parse@normalized
?The text was updated successfully, but these errors were encountered: