ms-intune-app-sdk-android icon indicating copy to clipboard operation
ms-intune-app-sdk-android copied to clipboard

PopupWindow not respecting transparent background on intune only

Open swolfand opened this issue 3 years ago • 3 comments

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

swolfand avatar Dec 08 '21 16:12 swolfand

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?

bannus avatar Dec 09 '21 19:12 bannus

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

swolfand avatar Dec 09 '21 19:12 swolfand

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.

codylund avatar Dec 29 '21 16:12 codylund