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

fix/95-barcode-proportion #96

Merged
merged 1 commit into from
Oct 20, 2019
Merged

fix/95-barcode-proportion #96

merged 1 commit into from
Oct 20, 2019

Conversation

johnferchermeli
Copy link
Contributor

@johnferchermeli johnferchermeli commented Oct 18, 2019

Description
Improve barcode proportions, by changing the proportion constrains. Height must be greater than 10% of the size of width and smaller than 20%.

The barcode should be something between the two images below.

Captura de Tela 2019-10-17 às 22 33 36

Captura de Tela 2019-10-17 às 22 33 53

Related Issue
resolve #95

Checklist

check with "x", if applied to your change

  • All methods associated with structs has func (s *struct) method() {} name style.
  • Wrote unit tests for new/changed features.
  • Updated docs/doc.go
  • Updated pkg/pdf/example_test.go
  • Updated README.md
  • New public methods has comments upside them explaining what it does
  • Executed go fmt github.com/johnfercher/maroto/... to format all files

@johnferchermeli
Copy link
Contributor Author

@SuddenGunter, resolving the issue of barcode. What do you think about?

Improve barcode proportions, by changing the proportion constrains. Height must be greater than 10% of the size of width and smaller than 20%.
@codecov-io
Copy link

codecov-io commented Oct 18, 2019

Codecov Report

Merging #96 into dev will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev      #96      +/-   ##
==========================================
+ Coverage   95.67%   95.69%   +0.02%     
==========================================
  Files           8        8              
  Lines         461      463       +2     
==========================================
+ Hits          441      443       +2     
  Misses         16       16              
  Partials        4        4
Impacted Files Coverage Δ
pkg/props/prop.go 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 87d45ab...31db529. Read the comment docs.

@johnferchermeli johnferchermeli changed the title Improve barcode proportions fix/95-barcode-proportion Oct 18, 2019
@johnfercher johnfercher merged commit b630a80 into dev Oct 20, 2019
@johnfercher johnfercher deleted the fix/95-barcode-proportion branch October 20, 2019 20:11
@johnfercher johnfercher mentioned this pull request Oct 22, 2019
felix pushed a commit to felix/maroto that referenced this pull request May 2, 2022
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.

None yet

3 participants