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

Themed Icon Support #78

Closed
CodElixer opened this issue Jun 14, 2022 · 6 comments · Fixed by #84
Closed

Themed Icon Support #78

CodElixer opened this issue Jun 14, 2022 · 6 comments · Fixed by #84
Assignees
Labels
enhancement New feature or request

Comments

@CodElixer
Copy link

CodElixer commented Jun 14, 2022

Hi Andrew!

It was really great for working together n helping to fix the Audio Overlapping Issue :)

As that major issue has been fixed, I would suggest you some minor enhancement for the app.

  1. As currently the BCR doesn't support Themed Icon, You can look into this for future updates.
    I've attached the screenshot, As you can see WhatsApp and other app icons are themed according to the System Theme.
    IMG_20220614_204017.jpg

  2. For me the app hasn't permission for the contact and that's why it wasn't saving contacts name in the recordings. When I provided the permission manually then It was saving the Contacts name with recordings.
    You can mention it in GitHub's Readme.md for manually providing Contacts Permission will record the contacts name in recordings or an option within the app that will specifically ask for the same.

Best Regards,
Yash

@chenxiaolong
Copy link
Owner

chenxiaolong commented Jun 14, 2022

Hmm, I'm not sure what would cause that. BCR has supported themed icons since the first version. It's working for me with the default Pixel launcher: Screenshot_20220614-133402 Does clearing the launcher's cache help at all?

You can mention it in GitHub's Readme.md for manually providing Contacts Permission will record the contacts name in recordings or an option within the app that will specifically ask for the same.

👍 I'll add it to the README for sure and think about whether to add an option to the app. EDIT: Maybe I can make it part of the initial permissions prompt (notifications required, microphone required, contacts optional).

@CodElixer
Copy link
Author

Hmm, I'm not sure what would cause that. BCR has supported themed icons since the first version. It's working for me with the default Pixel launcher: Screenshot_20220614-133402 Does clearing the launcher's cache help at all?

Ohh! And I thought it doesn't support because it never themed from the first install. Strange!!

EDIT: Maybe I can make it part of the initial permissions prompt (notifications required, microphone required, contacts optional).

Yeah, That'd be better 👍🏻

@CodElixer
Copy link
Author

CodElixer commented Jun 14, 2022

Tried clearing Cache of the Pixel Launcher but it didn't helped either!
Screenshot_20220614-233723_Settings.pngScreenshot_20220614-233808_Settings.png

@chenxiaolong
Copy link
Owner

Oh, I just thought of something: is your ROM based on Android 12 or 13? Themed icon support was officially added to Android in 13, but some ROMs make it work for third party apps on 12 by bundling icons for popular apps into the ROM. I wonder if that's the case here.

@CodElixer
Copy link
Author

CodElixer commented Jun 15, 2022

Oh, I just thought of something: is your ROM based on Android 12 or 13? Themed icon support was officially added to Android in 13, but some ROMs make it work for third party apps on 12 by bundling icons for popular apps into the ROM. I wonder if that's the case here.

Oops, It's Pixel Experience Plus ROM based on Android 12L #39 (comment).

chenxiaolong added a commit that referenced this issue Jun 15, 2022
…able

The contacts permission is used to add the contact name to the output
filenames. The permission prompt will only be shown the first time call
recording is enabled. If the user chooses to not allow the permission,
but wants to allow it later, they will have to do so via the system
settings.

Fixes: #78
Fixes: #80

Signed-off-by: Andrew Gunnerson <[email protected]>
@chenxiaolong
Copy link
Owner

I've added a section to the README about permissions and also added the contacts permission to the initial permissions prompt. That will be included in version 1.13.

@chenxiaolong chenxiaolong self-assigned this Jul 3, 2022
@chenxiaolong chenxiaolong added the enhancement New feature or request label Jul 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants