Skip to content

Fix outlining invoice item price with the header label 'price' #8454

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

Closed
wants to merge 1 commit into from
Closed

Fix outlining invoice item price with the header label 'price' #8454

wants to merge 1 commit into from

Conversation

joost-florijn-kega
Copy link
Contributor

@joost-florijn-kega joost-florijn-kega commented Feb 7, 2017

Fix the outlining of the invoice item price with the header label 'price' by having the same feed distance as used in the header.

In the header the label 'price' has a feed distance of 360:
https://github.com/magento/magento2/blob/develop/app/code/Magento/Sales/Model/Order/Pdf/Invoice.php#L99

This fixes issue #8453

Fix the outlining of the invoice item price with the header label 'price'.

In the header the label 'price' has a feed distance of 360:
https://github.com/magento/magento2/blob/develop/app/code/Magento/Sales/Model/Order/Pdf/Invoice.php#L99
@okorshenko okorshenko self-assigned this Feb 26, 2017
@okorshenko okorshenko added this to the February 2017 milestone Feb 26, 2017
@maksek maksek modified the milestones: February 2017, March 2017 Mar 1, 2017
@okorshenko okorshenko added this to the March 2017 milestone Mar 1, 2017
@okorshenko
Copy link
Contributor

@joost-florijn-kega thank you for your contribution. Unfortunately, we are not accepting PRs to 2.1 branch for now. Delivery procedure to patch releases is more complicated. We are working on establishing this process for 2.1 and 2.0 branches. Looks like this issue exists in develop branch as well. If you would like to fix this issue in develop branch, please, create new pull request to develop.
Closing this PR for now. We will consider backporting of this fix to 2.1.x release once it will be fixed in develop branch.

@okorshenko okorshenko closed this Mar 14, 2017
@magento magento locked and limited conversation to collaborators Mar 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants