Skip to content

docs: clarify block_text font parameter#734

Draft
Mirochill wants to merge 1 commit into
python-escpos:masterfrom
Mirochill:docs-731-clarify-block-text-font
Draft

docs: clarify block_text font parameter#734
Mirochill wants to merge 1 commit into
python-escpos:masterfrom
Mirochill:docs-731-clarify-block-text-font

Conversation

@Mirochill

Copy link
Copy Markdown

Description

Clarifies the block_text() documentation so the font parameter is described as a column-width lookup hint when columns is omitted, not as a command that switches the active printer font. The changelog also notes the documentation clarification.

Closes #731

Tested with

Not applicable; documentation-only change.

Validation:

  • git diff --check HEAD~1..HEAD
  • Static review of src/escpos/escpos.py and CHANGELOG.rst
  • Tests not run locally

@belono belono left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hi @Mirochill !

This clarification is very welcome and necessary, as seen in #731 .

You can remove the "Draft" label and leave the pull request open for @patkan to review and integrate.

Thanks!

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.

block_text does not set font on POS-5890

2 participants