Skip to content

Bugfix/427 back button doesnt work when activity launched#429

Merged
Canato merged 10 commits into
CanHub:mainfrom
Devenom1:bugfix/427-back-button-doesnt-work-when-activity-launched
Aug 29, 2022
Merged

Bugfix/427 back button doesnt work when activity launched#429
Canato merged 10 commits into
CanHub:mainfrom
Devenom1:bugfix/427-back-button-doesnt-work-when-activity-launched

Conversation

@Devenom1

Copy link
Copy Markdown
Contributor

close #427

Bug:

Clicking back on a screen that shows an alert dialog with Camera and Gallery would do nothing.

Cause:

The dialog is non dismiss-able so the back button is is ignored.

Reproduce

Open the crop image screen which shows the non dismiss-able alert dialog with the options Camera and Gallery.
Click back and nothing happens.

How the bug was solved:

Added an OnKeyListener to the non dismiss-able alert dialog to detect back key pressed. On back key detected the onBackPressed is triggered.

Also fixed the issue where on cancelling/pressing back when intending not to edit/crop the image it would open a screen with an empty image view.

@Devenom1
Devenom1 requested a review from a team as a code owner August 28, 2022 14:26
@Canato
Canato merged commit 1156433 into CanHub:main Aug 29, 2022
@ghost

ghost commented Sep 8, 2022

Copy link
Copy Markdown

@Canato When will be new version updated with these fixes.

@ghost

ghost commented Sep 9, 2022

Copy link
Copy Markdown

@Canato Thanks for the instant release of the new version with fixes.

@Canato

Canato commented Sep 9, 2022

Copy link
Copy Markdown
Member

Ohh my message yesterday didn't whent!

But thanks for the reminder. We need to find a way to automate snapshot generations until we do a release.
Was my bad that I forgot before!

@devpathare

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] - Back button dosen't work when activity launched.

2 participants