Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

My attempt at replacing fancyhdr with scrlayer-scrpage #174

Merged
merged 9 commits into from
Feb 7, 2021
Merged

My attempt at replacing fancyhdr with scrlayer-scrpage #174

merged 9 commits into from
Feb 7, 2021

Conversation

khaveesh
Copy link
Contributor

I tested and found no visual difference between the files produced with fancyhdr and scrlayer-scrpage. Now, the warnings issued by LuaLaTeX are silenced. Feel free to improve upon my code.

I also fixed the ifxetex & ifluatex legacy package warnings and replaced the complicated if pdftex checks with a simpler macro.

eisvogel.tex Outdated
\usepackage{ifxetex,ifluatex}
\ifnum 0\ifxetex 1\fi\ifluatex 1\fi=0 % if pdftex
\usepackage{iftex}
\ifpdftex
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you leave these lines as they are? I'm merging with the default template regularly and want to keep merge conflicts to a minimum.

It would be best to include these lines in the pandoc default template and make a pull request.

eisvogel.tex Outdated
@@ -51,6 +51,7 @@
$endif$
%
\documentclass[
version=last,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line can be removed since version=last is the default.

eisvogel.tex Outdated
@@ -74,7 +75,7 @@
$for(classoption)$
$classoption$$sep$,
$endfor$
,captions=tableheading
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch!

@khaveesh
Copy link
Contributor Author

khaveesh commented Feb 2, 2021

@Wandmalfarbe, I have made all the changes you requested for. I think you can merge it now.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants