Notification Expand arrow issues

wasdennnoch / AndroidN-ify

[Discontinued] An Xposed module to use features introduced in Android N on your LP+ Device

265 stars 93 forks source link

Notification Expand arrow issues #1091

Open Freewander10 opened 7 years ago

Freewander10 commented 7 years ago

When clicking the expand arrow on a notification, the notification doesn't expand or collapse, it just gets opened instead. The arrows only seem to work on whatsapp notifications, but only when the notification is from a single chat. UPDATE: Fixed.
The arrow sometimes faces the wrong direction on some devices.

Maxr1998 commented 7 years ago

I noticed that as well, it's pretty annoying. I am not sure if the touch targets are just to small or if it's something else, but I'll look into it.

Freewander10 commented 7 years ago

I tried it on a MM based ROM and it worked every time. Maybe the bug is ROM specific?

Maxr1998 commented 7 years ago

I'm on stock MM and still have it.. Also, I found the reason/one of the reasons. Currently fixing.

Maxr1998 commented 7 years ago

First issue should be fixed now.

Freewander10 commented 7 years ago

Yea. It's working as intended on build 730. Thanks bro 👍

Maxr1998 commented 7 years ago

I'd like to still monitor this issue regarding the wrong arrow direction. Also, I want to know why exactly mIcon was null and broke the if clause.

Freewander10 commented 7 years ago

Let me know if there's anything I can do to help. Also, see if you can get it to animate (wasdennnoch removed the animation as it was causing the arrows to not work any at all). I don't mind it being removed because having no animation is better than having no action, but having it back would be a plus. Maybe there's a workaround? You can check #825 for reference.