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

[FEAT] Optional maxWidth prop for barcode maximum width #58

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

Conversation

crherman7
Copy link

Description

There is no maximum width prop that would specify the maximum width of the barcode. There is a width prop but that specifies the width of each individual block in the barcode. With a use case of having dynamic values being shown as a barcode but a maximum width of a parent container the barcode would potentially overflow the parent container.

Solution

Adds optional prop - maxWidth of the barcode and determines if the barcode width exceeds the maxWidth; if the maxWidth is exceeded it then calculates the barcode block width based uponn the encoded data length and maxWidth.

@crherman7 crherman7 changed the title [FEAT] Optional maxWidth prop for barcode maximum width [FEAT][DO NOT MERGE] Optional maxWidth prop for barcode maximum width May 12, 2020
Adds optional prop - maxWidth of the barcode and determines if the
barcode width exceeds the maxWidth; if the maxWidth is exceeded it
then calculates the barcode block width based uponn the encoded
data length and maxWidth.
@crherman7 crherman7 force-pushed the feat/barcode_max_width branch from 4d523a6 to c3fb028 Compare May 12, 2020 22:27
@crherman7 crherman7 changed the title [FEAT][DO NOT MERGE] Optional maxWidth prop for barcode maximum width [FEAT] Optional maxWidth prop for barcode maximum width May 12, 2020
@danmcmillen
Copy link

@wonsikin is this something that could be looked at?

@danmcmillen danmcmillen mentioned this pull request Jul 16, 2020
Closed
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.

2 participants