Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

lscapeenhanced #1578

Closed
Rimole opened this issue Dec 7, 2024 · 2 comments
Closed

lscapeenhanced #1578

Rimole opened this issue Dec 7, 2024 · 2 comments

Comments

@Rimole
Copy link

Rimole commented Dec 7, 2024

There is: #166
And there is: https://www.ctan.org/pkg/lscapeenhanced

lscapeenhanced seems to fix the bug without breaking anything. (Does it?)

Would it be possible to merge lscapeenhanced into lscape?

@u-fischer
Copy link
Member

lscapeenhanced seems to fix the bug without breaking anything

Didn't you read the documentation? lscapeenhanced doesn't "fix" #166. \textheight is still (as with lscape) unchanged. But it additionally also leaves \textheight unchanged. Personally I doubt that users really want that:

\documentclass{article}
\usepackage{lscapeenhanced}
\begin{document}
\SetKeys[lscapeenhanced]{lscape=false}
\begin{landscape}
\noindent\rule{\linewidth}{\textheight}
\end{landscape}

\begin{landscape}
\noindent\rule{\textwidth}{\textheight}
\end{landscape}

\SetKeys[lscapeenhanced]{lscape=true}
\begin{landscape}
\noindent\rule{\textwidth}{\textheight}
\end{landscape}

\begin{landscape}
\noindent\rule{\linewidth}{\textheight}
\end{landscape}

\end{document}

The first two pages are the original, the following four pages with the lscapeenhanced patch:

image

@Rimole
Copy link
Author

Rimole commented Dec 22, 2024

So: "it depends". For the code below, I need \usepackage{lscapeenhanced}, otherwise "TEXT" is placed at the wrong place.
But because of the "it depends", it makes sense to not merge it.

\documentclass{article}
\usepackage{lscape}
\usepackage{lscapeenhanced}
\usepackage{atbegshi}
\AddToHook{shipout/before}{\AtBeginShipoutUpperLeft{%
\put(0sp,-\dimexpr 1in+\voffset+\topmargin+\headheight+\headsep+\footskip+\textheight\relax){TEXT}%
}}
\begin{document}
\begin{landscape}
landscape
\end{landscape}
\end{document}

@Rimole Rimole closed this as completed Dec 22, 2024
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

No branches or pull requests

2 participants