search

microsoftconnect / ms-intune-app-sdk-android

Intune App SDK for Android enables data protection features and mobile app management via Microsoft Intune
46 stars 18 forks source link

PopupWindow not respecting transparent background on intune only #88

Open swolfand opened 2 years ago

swolfand commented 2 years ago

Describe the bug: At Slack we use the Android framework PopupWindow for user education, and on intune builds only we see that the transparent background of our 9 patch drawables aren't respected (see image) does intune do any transformation of these classes?

To Reproduce Steps to reproduce the behavior:

  1. Install InTune build (From appstore is fine)
  2. Register with a Microsoft account
  3. Sign-in to an org
  4. Go to any channel
  5. Tap the Input field to bring up the toolbar
  6. Look at the tooltip shown for formatting

Expected behavior: The tooltip should be shown as in the regular builds, no additional background/border

Screenshots and logs: image

Smartphone (please complete the following information): -Android Emulator Pixel 2 API 30

Intune SDK 7.6.0

bannus commented 2 years ago

Hi @swolfand, thanks for the report. We're tracking this internally with #12812733.

Which version of the Company Portal did you use? Can you confirm it reproduces with the latest version of the Company Portal?

swolfand commented 2 years ago

Hi @swolfand, thanks for the report. We're tracking this internally with #12812733.

Which version of the Company Portal did you use? Can you confirm it reproduces with the latest version of the Company Portal?

I'm using 5.0.5333 I just reproduced it today with the latest version

codylund commented 2 years ago

Digging into this issue, I can reproduce locally in a test app without the Company Portal installed. My guess is that we are losing some default theming when the Build Plug-in rewrites PopupWindow to MAMPopupWindow, but I haven't root caused the issue.

@swolfand, one workaround I discovered via local testing is to explicitly set a transparent background on the PopupWindow instance, e.g.

popWindow.setBackgroundDrawable(new ColorDrawable(Color.TRANSPARENT));

We will continue tracking this issue and will hopefully investigate in more depth soon. However, this workaround should fix the issue for Slack in the short term.