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

Combine purchaser_name together with purchaser_lines #83

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

voxik
Copy link
Contributor

@voxik voxik commented Mar 20, 2023

Quite often the purchaser name looks like:

John Doe,
Foo Inc.

Where the invoice is prepared for John Doe from Foo Inc. company. If this is the case, the purchaser_name becomes too long and ovelaps the `purchaser_lines.

The overlapping can be avoided by combining the purchaser_name and purchaser_lines, outputting just the first line in bold and treating the rest just as an address. This should provide the greatest flexibility and it is actually extension to #77.

Before:

obrazek

After:

obrazek

Quite often the purchaser name looks like:

~~~
John Doe,
Foo Inc.
~~~

Where the invoice is prepared for John Doe from Foo Inc. company. If
this is the case, the `purchaser_name` becomes too long and ovelaps the
`purchaser_lines.

The overlapping can be avoided by combining the `purchaser_name` and
`purchaser_lines`, outputting just the first line in bold and treating
the rest just as an address. This should provide the greatest
flexibility and it is actually extension to strzibny#77.
@strzibny
Copy link
Owner

strzibny commented Apr 1, 2023

I like the idea. Maybe the provider should be treated the same?

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.

None yet

2 participants