Skip to content
This repository has been archived by the owner on May 10, 2021. It is now read-only.

Use system font when printing. #222

Merged
merged 1 commit into from
Feb 13, 2018
Merged

Use system font when printing. #222

merged 1 commit into from
Feb 13, 2018

Conversation

tosbourn
Copy link
Contributor

Information sheet prints with an error on second heading: 'Consentf orm' rather than 'Consent form'

Uses a system font (Verdana) when printed. This required some sizing changes to be made to accommodate Verdana being larger when used than ProximaNova.

Some other small changes were made:

  • A tick was being used for "Barnardo's", this has been replaced with an apostrophe
  • Padding was being added to editable text, when printed editable doesn't make sense and the padding looked off when characters were no longer aligned.

This solution has been tested against Kyocera TASKalfa 5052ci KM4C35C6 which was the printer that was reported to cause this error.

Example

This example shows the improved apostrophe and no padding on text inputs.

screen shot 2018-02-12 at 16 33 33

This required some sizing changes to be made to accomodate Verdana
being larger when used than ProximaNova.

Some other small changes were made:

* A tick was being used for "Barnardo's", this has been replaced
with an apostrophe
* Padding was being added to editable text, when printed editable
doesn't make sense and the padding looked off when characters were
no longer aligned.
@tosbourn tosbourn requested a review from joelcarr February 12, 2018 16:34
Copy link
Contributor

@joelcarr joelcarr left a comment

Choose a reason for hiding this comment

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

🖨

@tosbourn tosbourn merged commit e7d1fe6 into master Feb 13, 2018
@tosbourn tosbourn deleted the printing-issues branch February 13, 2018 11:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants