Skip to content

"Open settings" button in the Camera permission rationale dialog#445

Merged
AdamGrzybkowski merged 3 commits intotrunkfrom
adam/422_permision_rationale
Nov 19, 2024
Merged

"Open settings" button in the Camera permission rationale dialog#445
AdamGrzybkowski merged 3 commits intotrunkfrom
adam/422_permision_rationale

Conversation

@AdamGrzybkowski
Copy link
Copy Markdown
Contributor

Closes #422

Description

The camera permission rationale dialog lacked the option to open the App settings directly. This PR changes that.

Unfortunately, I couldn't find a way to launch the permission settings directly.

open_settings.mp4

Testing Steps

  1. Go to /demo-app/AndroidManifest.xml and uncomment the camera permission
  2. Start the demo app
  3. Try to take a picture
  4. Decline the persission
  5. Rationale Dialog should be shown - tap the open settings
  6. Confirm the Demo App settings were opened

@AdamGrzybkowski AdamGrzybkowski added the [Feature] Gravatar-Quickeditor Gravatar Quick Editor module label Nov 18, 2024
@wpmobilebot
Copy link
Copy Markdown
Collaborator

wpmobilebot commented Nov 18, 2024

📲 You can test the changes from this Pull Request in Gravatar Demo by scanning the QR code below to install the corresponding build.
App Name Gravatar Demo
Commit77f091c
Direct Downloadgravatar-demo-prototype-build-pr445-77f091c.apk

Copy link
Copy Markdown
Contributor

@hamorillo hamorillo left a comment

Choose a reason for hiding this comment

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

Looks good to me! I left some comments about the removed ./idea files.

I couldn't find a way to launch the permission settings directly

Same here. I couldn't find anything, so I think that works. I'll pay attention to other apps and see if any can navigate directly to the permissions screen.

🚀

@@ -1,44 +0,0 @@
<component name="InspectionProjectProfileManager">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤔 I have some doubt about removing this file based on this

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I know that I didn't make any changes to our project config that should be shared, yet those files were modified. If this is some automated stuff by AS, I see no reason to share it. I haven't dug deep into it though...

@@ -1,6 +0,0 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤔 I have some doubt about removing this file based on this however taking a look at gitignore.io they would exclude it.

@AdamGrzybkowski AdamGrzybkowski force-pushed the adam/422_permision_rationale branch from 62dc639 to 77f091c Compare November 19, 2024 10:24
@AdamGrzybkowski AdamGrzybkowski merged commit b5c15ae into trunk Nov 19, 2024
@AdamGrzybkowski AdamGrzybkowski deleted the adam/422_permision_rationale branch November 19, 2024 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Feature] Gravatar-Quickeditor Gravatar Quick Editor module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve the permission rationale dialog UI

3 participants