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

Revert DYMO label margins to values prior to DYMO tape printing being added #132

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

Conversation

ribbons
Copy link

@ribbons ribbons commented Mar 11, 2024

When DYMO tape printing support was added in ed195b5 and 6f49c6e, the label left_right margin was changed from 100 to 367 and the bottom_top margin from 525 to 1.

In my limited testing (I currently only have a LabelWriter 450 and 1.4 x 3.5" Large Address labels to test it with), these values cause the label content to be offset both vertically and horizontally with respect to the label. For instance, after selecting the correct media, printing the test page gives me this result (next label manually ejected too for clarity):

before

The commit in this PR restores the values to what they were prior to ed195b5 and gives me the following result from the test page:

after

This was initially a bit of a head-scratcher for me because I hadn't realised that the margins are stored in the state file and only updated when media is selected (or re-selected).

When DYMO tape printing support was added in ed195b5 and 6f49c6e,
the label left/right margin was changed from 100 to 367 and the
bottom/top margin from 525 to 1.  This causes the label content to be
offset both vertically and horizontally with reference to the label.

The margin values are stored in the state file along with the other
media details so it is necessary to re-select the media size (or remove
the state file) for this change to take effect.
@michaelrsweet michaelrsweet self-assigned this Mar 14, 2024
@michaelrsweet michaelrsweet added bug Something isn't working priority-medium labels Mar 14, 2024
@michaelrsweet michaelrsweet added this to the Stable milestone Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority-medium
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants