Skip to content
This repository has been archived by the owner on Jan 16, 2018. It is now read-only.

Style errors on invoice #7

Open
aisopuro opened this issue Aug 16, 2016 · 0 comments
Open

Style errors on invoice #7

aisopuro opened this issue Aug 16, 2016 · 0 comments

Comments

@aisopuro
Copy link
Contributor

There are some style problems on the invoice.

  1. If a line has a long description, the wrapped text is not in line with the start of the text
    image
  2. If the billing part goes over to the next page, it overlaps with the header.
    image
aisopuro added a commit to aisopuro/l10n_fi_invoice that referenced this issue Aug 17, 2016
@aisopuro
[FIX] Fix style errors in the invoice:
1e590e3 
- The invoice lines wrapped text poorly for long line descriptions.
  The text is now in line with the product name.
- The margins for the header and the start of the invoice were
  pretty large. These have been decreased somewhat. Various
  other margins and min-heights have also been adjusted.
- The bank statement part, if it propagated to the following page,
  overlapped with the header of that page. The header has been
  properly separated to a separate template, so that the page and
  header are not rendered in the same space for PDF reports.
- The sum and taxes part of the invoice is forced to stay in
  one part so it doesn't split on page breaks.

Closes avoinsystems#7
@aisopuro aisopuro mentioned this issue Aug 17, 2016
Merged
aisopuro added a commit to aisopuro/l10n_fi_invoice that referenced this issue Aug 18, 2016
@aisopuro
[FIX] Forgot some references to doc (avoinsystems#7)
fbe6fff
aisopuro added a commit to aisopuro/l10n_fi_invoice that referenced this issue Aug 18, 2016
@aisopuro
[FIX] Add some bottom margin to avoid bank statement touching the bot…
e360443 
…tom (avoinsystems#7)
mlaitinen added a commit that referenced this issue Aug 18, 2016
@mlaitinen
Merge pull request #10 from aisopuro/7-style-fixes
dcbf0c9 
[FIX] Fix style errors in the invoice (#7):
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@aisopuro