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

Android - close buttons (chat and settings) - #1763 fix #1876

Merged
merged 3 commits into from
Jun 15, 2021

Conversation

ngocdh
Copy link
Contributor

@ngocdh ngocdh commented Jun 14, 2021

Fix for #1763: add close button for dialogs in Android.

src/chatdlg.cpp Outdated Show resolved Hide resolved
src/clientsettingsdlg.cpp Outdated Show resolved Hide resolved
@pljones pljones changed the title Android - chatdlg close button - #1763 fix Android - close buttons (chat and settings) - #1763 fix Jun 15, 2021
@ann0see
Copy link
Member

ann0see commented Jun 15, 2021

I don’t have an Android device at hand at the moment, but can anyone test it?

@pljones
Copy link
Collaborator

pljones commented Jun 15, 2021

Loaded up, connected, got feedback that didn't get detected, disconnected, checked in settings and feedback detection was enabled, closed settings using the menu. Checked Chat - closed using the menu okay.

(Android Moto G9 Power - the vertical sizing does need to be addressed still, my screen was too short.)

Copy link
Member

@ann0see ann0see left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok. So happy to approve

@ann0see ann0see merged commit 40c794d into jamulussoftware:master Jun 15, 2021
@ann0see
Copy link
Member

ann0see commented Jun 15, 2021

CHANGELOG: Add close button on Android to enhance UX

@hoffie hoffie added this to the Release 3.8.1 milestone Jun 15, 2021
@ngocdh
Copy link
Contributor Author

ngocdh commented Jun 16, 2021

Great thanks team :)

@ngocdh ngocdh deleted the patch-3 branch June 16, 2021 04:50
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.

4 participants