Skip to content

Conversation

@pgray
Copy link

@pgray pgray commented Sep 19, 2024

I tried to implement #3 using the graphicx package

It seems to work well in my testing, though looks a bit better if the minipage sections are top aligned.

I'm unsure how to add graphicx as a conditional dependency.

Please let me know if there's a different approach or where to document the required usepackage{graphicx} if someone wants to set a logo.

\newcommand{\setcurrency}[1]{\def \INV@text@currency {\ #1}}
\newcommand{\setvattext}[1]{\def \INV@text@vat {#1}}
\newcommand{\setaccounttext}[1]{\def \INV@text@accountnum {#1}}
\newcommand{\setlogo}[1]{\def \INV@content@logo} {#1}}
Copy link

Choose a reason for hiding this comment

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

I think there is an extra } at the end of \INV@content@logo which causes the error You can't use macro parameter character #' in vertical mode.`

@podiki
Copy link

podiki commented Aug 11, 2025

This would be a great addition! I tried it out but doesn't seem like the minipage layout is working for me, I get the logo going across the entire page and behind text. The test image was 450x150, not particularly large but it is stretched out to fit the page width.

Edit: something like [width=0.8\textwidth] in the includegraphics works for me, though also don't like the alignment happening between the textboxes. Adding some vertical space after the logo, like \vspace{2.5pc} in my case works.

Anyway, perhaps specifying a required logo size or adding some parameters to the \setlogo command to allow spacing/positioning, will work better in general.

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