apptentive / apptentive-android

Apptentive Android SDK
http://www.apptentive.com
BSD 3-Clause "New" or "Revised" License
65 stars 64 forks source link

Cannot figure out how to configure Apptentive "About" dialog buttons. #235

Closed elliottj-accolade closed 2 years ago

elliottj-accolade commented 2 years ago

Apptentive v. 5.8 Android 12, Pixel 5

Hello,

We have an issue where the Apptentive buttons on the apptentive_about fragment are not displaying. We have a complicated theme that supports both dark and light modes. Unfortunately this has made it very hard to diagnose exactly which override is causing the problem.

On an empty project, buttons appear no issues: Screenshot_20220118-154834

On our project, buttons do not appear: Screenshot_20220118-154814

I've browsed around in the layout file and found that the buttons both use a style of Apptentive.Widget.Button.Borderless, and a textAppearance of Apptentive.TextAppearance.Body1. I haven't had any luck in overwriting any of the values in either of those themes.

Do you have any suggestions of what values I can override to change that? I notice in an empty project that setting colorPrimary on the ApptentiveThemeAbout works, but it didn't work in the heavily themed project.

BTW, I acknowledge this smells like a fishing expedition. I'm hoping for a direction other than what I mentioned above.

CaseyApptentive commented 2 years ago

Hi @elliottj-accolade. Great question. Sometimes that section does have trouble with more complex styles, especially in dark mod.

My immediate thought is that you may want to simply remove the (i) info button. Would that fit your needs? Details on how to do that:


You'd need to override the theme res/values/themes-apptentive-override.xml. This will hide the (i) button in both Message Center and Surveys.

<?xml version="1.0" encoding="utf-8"?>
<resources>
    <style name="Apptentive.Style.Widget.ImageButton.Info">
        <item name="android:visibility">invisible</item>
    </style>
</resources>
elliottj-accolade commented 2 years ago

Hi @CaseyApptentive, that works for us. Thank you for the suggestion!

CaseyApptentive commented 2 years ago

Happy to help. Let me know if you need anything else.