Skip to content
This repository has been archived by the owner on Oct 10, 2020. It is now read-only.

feat: added cancel for ux #198

Merged
merged 6 commits into from Apr 20, 2020
Merged

feat: added cancel for ux #198

merged 6 commits into from Apr 20, 2020

Conversation

iRaySpace
Copy link
Contributor

@iRaySpace iRaySpace commented Mar 24, 2020

There are users that doesn't know back button. Added the Cancel button for UX purposes. Only modified for Android. Haven't done for iOS
Cancel

@devtronic devtronic changed the base branch from master to 3.x April 12, 2020 12:22
@devtronic
Copy link
Member

Thanks for contributing 🙂

Please merge the current master branch in your branch.
I also added few notes to your changes

@iRaySpace
Copy link
Contributor Author

iRaySpace commented Apr 16, 2020

Hello, can you check? Not sure, if I did right.

@devtronic devtronic self-requested a review April 16, 2020 07:10
@ccfiel
Copy link

ccfiel commented Apr 17, 2020

@devtronic when will this merge? :)

@devtronic
Copy link
Member

I'll review the changes today

@ccfiel
Copy link

ccfiel commented Apr 17, 2020

@devtronic thanks for the update 😊

@devtronic
Copy link
Member

@iRaySpace I can't build the project since there is a variable getting defined twice. I added a comment on the corresponding line in the BarcodeScannerActivity.kt. After you fixed that I'll merge the PR.

@iRaySpace
Copy link
Contributor Author

Hello @devtronic Thank you, and also I have put up the changes.

@devtronic devtronic self-requested a review April 20, 2020 20:48
@devtronic devtronic marked this pull request as draft April 20, 2020 20:48
@devtronic devtronic removed their request for review April 20, 2020 20:48
@devtronic devtronic marked this pull request as ready for review April 20, 2020 20:48
@devtronic devtronic merged commit a6af2f5 into mintware-de:3.x Apr 20, 2020
@devtronic devtronic mentioned this pull request May 8, 2020
devtronic added a commit that referenced this pull request May 8, 2020
* feat: added cancel for ux (#198)

* feat: added cancel for ux

* re(feat): apply cancel functionality to merged

* fix: return true

* fix(syntax): flashButton and cancelButton

* fix: syntax

* Changelog updated

* add fix for autofocus with logging

* add comment and remove log for merge on master project

* Update the 3.x branch (#229)

* Update issue templates

* Update UPGRADE.md

* Fix camera selection in Android.

* CHANGELOG updated

* Release prepared

Co-authored-by: Ivan Ray Altomera <iRaySpace@users.noreply.github.com>
Co-authored-by: Vincenzo <vincenzo.salerno@live.it>
Co-authored-by: GabrieleVolpato <6254043+GabrieleVolpato@users.noreply.github.com>
devtronic added a commit that referenced this pull request Sep 14, 2020
* 3.0.1 (#237)

* feat: added cancel for ux (#198)

* feat: added cancel for ux

* re(feat): apply cancel functionality to merged

* fix: return true

* fix(syntax): flashButton and cancelButton

* fix: syntax

* Changelog updated

* add fix for autofocus with logging

* add comment and remove log for merge on master project

* Update the 3.x branch (#229)

* Update issue templates

* Update UPGRADE.md

* Fix camera selection in Android.

* CHANGELOG updated

* Release prepared

Co-authored-by: Ivan Ray Altomera <iRaySpace@users.noreply.github.com>
Co-authored-by: Vincenzo <vincenzo.salerno@live.it>
Co-authored-by: GabrieleVolpato <6254043+GabrieleVolpato@users.noreply.github.com>

* Update issue templates

* Scan Options Descriptions correction. (#270)

Co-authored-by: Ivan Ray Altomera <iRaySpace@users.noreply.github.com>
Co-authored-by: Vincenzo <vincenzo.salerno@live.it>
Co-authored-by: GabrieleVolpato <6254043+GabrieleVolpato@users.noreply.github.com>
Co-authored-by: Muzammil Bilwani <muzammil.bilwani@hotmail.com>
devtronic added a commit that referenced this pull request Sep 14, 2020
* 3.0.1 (#237)

* feat: added cancel for ux (#198)

* feat: added cancel for ux

* re(feat): apply cancel functionality to merged

* fix: return true

* fix(syntax): flashButton and cancelButton

* fix: syntax

* Changelog updated

* add fix for autofocus with logging

* add comment and remove log for merge on master project

* Update the 3.x branch (#229)

* Update issue templates

* Update UPGRADE.md

* Fix camera selection in Android.

* CHANGELOG updated

* Release prepared

Co-authored-by: Ivan Ray Altomera <iRaySpace@users.noreply.github.com>
Co-authored-by: Vincenzo <vincenzo.salerno@live.it>
Co-authored-by: GabrieleVolpato <6254043+GabrieleVolpato@users.noreply.github.com>

* Update issue templates

* Scan Options Descriptions correction. (#270)

* Update README.md

Update Kotlin plugin installation link, add protobuf-gradle-plugin dependency for Gradle build

Co-authored-by: Julian Finkler <julian@mintware.de>
Co-authored-by: Ivan Ray Altomera <iRaySpace@users.noreply.github.com>
Co-authored-by: Vincenzo <vincenzo.salerno@live.it>
Co-authored-by: GabrieleVolpato <6254043+GabrieleVolpato@users.noreply.github.com>
Co-authored-by: Julian Finkler <julian@developer-heaven.de>
Co-authored-by: Muzammil Bilwani <muzammil.bilwani@hotmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Did the author think about it and add a back button! Many phones do n’t have physical buttons
3 participants