openfoodfacts / smooth-app

The new Open Food Facts mobile application for Android and iOS, crafted with Flutter and Dart

Home Page:https://world.openfoodfacts.org/open-food-facts-mobile-app?utm_source=off&utf_medium=web&utm_campaign=github-repo

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

(Easily dismissed) Grey screen in some cases for receipt scanning

teolemon opened this issue · comments

What

  • (Easily dismissed) Grey screen in some cases for receipt scanning

Screenshot

image

@teolemon Technically it's not "Receipt Scanning", it's "Barcode Scanning" when adding a product to a receipt or price tags.

From my own experience, I assume it happens when going back to the app.
Probably related to camera being reactivated and all the tap dancing about it, especially given that we now have potentially 2 barcode readers at the same time - the "home" one and this "prices" one.

@g123k Are there plans to get rid of the current bottom bar mechanism? TBH it's getting a bit cryptical - the bottom bar is on some pages only, for an untold reason - and it causes a lot of problem about the camera being deactivated/reactivated (as opposed to "in the current page there is / isn't a barcode reader, therefore I do / don't need a camera").
I guess we could keep the bottom bar while not pretending to display 3 pages simultaneously, what do you think?

@teolemon Technically it's not "Receipt Scanning", it's "Barcode Scanning" when adding a product to a receipt or price tags.

From my own experience, I assume it happens when going back to the app. Probably related to camera being reactivated and all the tap dancing about it, especially given that we now have potentially 2 barcode readers at the same time - the "home" one and this "prices" one.

@g123k Are there plans to get rid of the current bottom bar mechanism? TBH it's getting a bit cryptical - the bottom bar is on some pages only, for an untold reason - and it causes a lot of problem about the camera being deactivated/reactivated (as opposed to "in the current page there is / isn't a barcode reader, therefore I do / don't need a camera"). I guess we could keep the bottom bar while not pretending to display 3 pages simultaneously, what do you think?

In the POC, there's still the bottom bar, but I have implemented the camera with a different algorithm.
Currently, in the app, we use the default mode = autostart. It forces the camera to be opened and to prevent edge cases, we have several if to close it, if we are on a different screen.

In the POC, I do things differently, where I manage manually when I want the camera.
But changing things in the current version brings us to months ago (bugs & co).
If we have a plan B, I wouldn't be against.