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

129 - demo app redesign #182

Merged
merged 47 commits into from
Jun 5, 2024
Merged

129 - demo app redesign #182

merged 47 commits into from
Jun 5, 2024

Conversation

VincentKobz
Copy link
Collaborator

  • Redesign of the enioka scan demo-app.
  • Comes bundled with all OSS SDKs.

@VincentKobz VincentKobz self-assigned this May 13, 2024
@VincentKobz VincentKobz force-pushed the 129-demo-app-redesign branch 2 times, most recently from 4b13101 to 4479eb5 Compare May 15, 2024 13:54
@VincentKobz VincentKobz requested a review from DaSpood May 15, 2024 13:54
@VincentKobz VincentKobz force-pushed the 129-demo-app-redesign branch 13 times, most recently from c98e6db to 5b288b4 Compare May 16, 2024 13:43
@VincentKobz VincentKobz force-pushed the 129-demo-app-redesign branch 8 times, most recently from 0584bd6 to 2d5a654 Compare May 23, 2024 12:21
@VincentKobz VincentKobz force-pushed the 129-demo-app-redesign branch from 7a54329 to 3ce70a2 Compare May 24, 2024 14:55
@VincentKobz VincentKobz force-pushed the 129-demo-app-redesign branch from 3ce70a2 to a19abe9 Compare May 24, 2024 15:01
@VincentKobz VincentKobz force-pushed the 129-demo-app-redesign branch from 9023b7d to 6dc6a9a Compare May 27, 2024 09:54
@DaSpood
Copy link
Collaborator

DaSpood commented May 28, 2024

Dernier feedback sur la partie graphique:

  • Dans la vue camera, avoir le texte des boutons "ZXing/Zbar" mentionner les deux ou passer de l'un a l'autre
  • Dans la vue camera, avoir le texte des boutons "ZXing" et "Pause" plus epais et avec un contour blanc pour qu'il reste lisible peu importe ce qui est a l'ecran (noir sur fond sombre arrive frequemment)
  • Dans la vue camera, si on y accede depuis la vue scan normale (si le setting "default to camera" est off) faire un retour arriere devrait renvoyer a la vue scan, pas au menu principal
  • Dans la vue settings, peut etre passer le bouton save en flottant ou sur la barre header car si on ne scrolle pas jusqu'en bas on peut ne pas comprendre qu'il faut sauvegarder ses changements
  • Pour le fichier de logs, mettre le mot "log" au debut du filename car selon les file browsers on ne voit pas le nom entier donc on ne devine pas forcement que ca nous demande un chemin pour les logs et pas autre chose
  • Dans les settings, peut etre un toggle pour changer entre le mode fill et le mode preserver l'aspect ratio si c'est faisable facilement
  • Faire en sorte que le nom de l'appli soit uniquement "enioka Scan" pour qu'il apparaisse en entier sous l'icone. On peut garder la precision "demo" pour la page store.

Sinon ca me parait bien, l'UI est tres clean

@VincentKobz VincentKobz force-pushed the 129-demo-app-redesign branch from d6a2a15 to 9592c26 Compare May 29, 2024 14:56
@DaSpood
Copy link
Collaborator

DaSpood commented May 31, 2024

  • Texte vide = EAN13 non traduit sous Allowed Symbology dans l'activity settings
  • ZXing / ZBar -> ZBar / ZXing pour que la position du switch corresponde a la position de l'API utilisee (actuellement, switch a gauche = API de droite)
  • La target disparait en mode paysage
  • Enable logging -> Enable scan history et rename le fichier de log_... -> scans_.... Si possible aussi convertir la timestamp en format lisible (yyyyMMdd_HHmmss) (dans la premiere colonne du CSV et le nom du fichier)

@VincentKobz VincentKobz force-pushed the 129-demo-app-redesign branch from e97a44c to 7d71b9c Compare June 3, 2024 09:34
@VincentKobz VincentKobz force-pushed the 129-demo-app-redesign branch from 82acd4b to 31a24a9 Compare June 4, 2024 12:24
@DaSpood DaSpood merged commit 044e344 into master Jun 5, 2024
2 checks passed
@DaSpood DaSpood deleted the 129-demo-app-redesign branch June 5, 2024 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants