Skip to content

Conversation

@aurelbof
Copy link

In this version a few lines have been changed to replace sectsty by titlesec.

The package titlesec [3] functions equivalently to sectsty, although it is a bit more complex and powerful (it is referenced a few times in the sectsty documentation [1], likewise titlesec mentions sectsty [2]).

The biggest reason for this change is that it gets rid of these annoying warnings when using sectsty (i have not found a way to suppress them when still keeping sectsty):

LaTeX Warning: Command \underbar  has changed
LaTeX Warning: Command \underline has changed

This is quite a pedantic reason for a change, I know. These warnings could be ignored with no consequence.

On the technical side: a few lines in iacrtrans.cls were changed, There is no direct \allsectionsfont equivalent in titlesec, but \section, \subsection, \subsubsection, \paragraph and \subparagraph are all of the sectional headings in the article document class (which iacrtrans herits from), so individually specifying each \titleformat is equivalent.

[1] https://distrib-coffee.ipsl.jussieu.fr/pub/mirrors/ctan/macros/latex/contrib/sectsty/sectsty.pdf
[2] https://ctan.ceremade.dauphine.fr/macros/latex/contrib/titlesec/titlesec.pdf
[3] https://ctan.org/pkg/titlesec

@kmccurley
Copy link

Note that the documentation for titlesec says that it is not guaranteed to be compatible with hyperref. That leads me to believe that titlesec may be too complicated a dependency just to solve this problem.

Another suggested fix is in this article where they suggest using

\usepackage{silence}
\WarningFilter[temp]{latex}{Command \underbar  has changed.} \WarningFilter[temp]{latex}{Command \underline  has changed.} 

This seems like a better solution but I confess that I haven't tried it.

@aurelbof
Copy link
Author

Right, I guess changing a whole dependency just for a warning is a little overkill.

The silence solution doesn't seem to work on my end (just \RequirePackage{silence} makes compilation run forever), maybe due to a conflict with hyperref (similar issue as in https://tex.stackexchange.com/questions/735915/cant-use-package-silence-with-hyperref-and-acronym).

Hyperref doesn't seem to create problems in my tests, metadata seems fine (toc in particular). But I guess there might be examples where it breaks down, so I understand if the PR doesn't get accepted. I'll just run with my fork for now :).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants