feat(barcode-scanning): pause and resume scan feature, cap 7

[bakai-dev-team]

Pull request checklist

Please check if your PR fulfills the following requirements:

• The changes have been tested successfully.
• A changeset has been created (npm run changeset).
• I have read and followed the pull request guidelines.

[matthiasschwarz]

Is this pull request related to #298?
If this is the case, this implementation would, at least for Android and the web, only pause and resume the barcode scan, but not freeze the camera preview.

[bakai-dev-team]

Is this pull request related to #298? If this is the case, this implementation would, at least for Android and the web, only pause and resume the barcode scan, but not freeze the camera preview.

Partially it resolves #298 too. We didn't test web version.

On Android

if pauseScan called it will skip all scanned results (will not resolve call) until resumeScan called

public void analyze(@NonNull ImageProxy imageProxy) {
        if (isPaused) {
            imageProxy.close();
            return;
        }
        ....
}

On iOS

It fill stop capture session (freeze) until resumeScan called

DispatchQueue.main.async {
          if let session = self.cameraView?.getCaptureSession() {
              if session.isRunning {
                  session.stopRunning()
              }
          }
}

[robingenz]

@bakai-dev-team It should freeze the camera view. Otherwise, this feature is useless since you can do the same in the webview (just remove the listener).

[bakai-dev-team]

@bakai-dev-team It should freeze the camera view. Otherwise, this feature is useless since you can do the same in the webview (just remove the listener).

On IOS it's already behaves as you mentioned.

How can we do the same on Android?

[robingenz]

@bakai-dev-team I haven't looked into it yet. It's part of the issue to find that out.

[bakai-dev-team]

@robingenz All done as you required.

Anyone who can check and confirm that it is works, you are welcome.

[erlanz]

@robingenz HI, could you please check PR, thanks

[robingenz]

PR is still failing.

[erlanz]

@robingenz Hello, fix lint, prettier. could you please check PR, thanks

[pkg-pr-new]

Open in StackBlitz

@capacitor-mlkit/barcode-scanning

npm i https://pkg.pr.new/capawesome-team/capacitor-mlkit/@capacitor-mlkit/barcode-scanning@300

@capacitor-mlkit/document-scanner

npm i https://pkg.pr.new/capawesome-team/capacitor-mlkit/@capacitor-mlkit/document-scanner@300

@capacitor-mlkit/face-detection

npm i https://pkg.pr.new/capawesome-team/capacitor-mlkit/@capacitor-mlkit/face-detection@300

@capacitor-mlkit/face-mesh-detection

npm i https://pkg.pr.new/capawesome-team/capacitor-mlkit/@capacitor-mlkit/face-mesh-detection@300

@capacitor-mlkit/selfie-segmentation

npm i https://pkg.pr.new/capawesome-team/capacitor-mlkit/@capacitor-mlkit/selfie-segmentation@300

@capacitor-mlkit/subject-segmentation

npm i https://pkg.pr.new/capawesome-team/capacitor-mlkit/@capacitor-mlkit/subject-segmentation@300

@capacitor-mlkit/translation

npm i https://pkg.pr.new/capawesome-team/capacitor-mlkit/@capacitor-mlkit/translation@300

commit: aa3b0a0

[erlanz]

@robingenz Hello, fix lint, prettier. could you please check PR, thanks

[robingenz]

Have you tested your implementation on all platforms?

[konradliebig]

What is still missing in this PR? This would be an awesome feature to have.

[robingenz]

Please fix the merge conflicts.

[robingenz]

Please revert those changes.

[robingenz]

Please revert those changes.

[konradliebig]

@erlanz This branch still has merge conflicts. Also why did you add the patch-package script, even though no package is patched? And dist should also not be part of the repo.