-
Notifications
You must be signed in to change notification settings - Fork 2
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
[WebComponents] Enh document capture margins #292
Conversation
This branch has been deployed to s3 / cloudfront. ✅ Preview URL for Smart Camera Web: |
Add height and width attributes
Reuse this._drawLandscapeImage for summited image when in landscape mode Refactor _drawLandscapeImage to accept scaleheight and width
This branch has been deployed to s3 / cloudfront. ✅ Preview URL for Smart Camera Web: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Handle Portrait Mode for documents like green card * Fix missing function error
This improves the capture of documents by adding extra margins to the submitted image.
https://app.shortcut.com/smileid/story/13030/add-extra-margins-when-capturing-documents-to-prevent-overcropping-on-web-component
Testing
Example App
The Example App can be used to test out changes made locally to the embed and it's dependencies.
To start testing locally with the embed
Copy the Sample env to example/.env in the example app and provide the relevant values
Build and start the embed server (it should be running on port 8000)
cd packages/embed npm install npm build npm start
Start the example app
cd example npm install npm start npm run dev