search

nitanmarcel / cinny-click-packaging

Yet another matrix client. Click packaging for locally running on Ubuntu Touch
Other
9 stars 7 forks source link

White screen on push notification tap when the app is already running #18

Open ezze opened 1 year ago

ezze commented 1 year ago

Describe the bug White screen when tapping on message push notification.

To Reproduce

Steps to reproduce the behavior:

  1. Open the app from menu or by clicking on push notification.
  2. Make the app inactive, e.g. run another app (don't close Cinny by swiping up or down).
  3. Send a message to device Matrix account from somewhere.
  4. Tap on new push notification.
  5. White screen as the result

Expected behavior App becomes active and you can read the received message.

Screenshots

image

Additional context Not really related to this issue but after migrating from Fluffy Chat it's a little bit annoying to receive a push notification for each message from the room. It would be really great to receive one push notification only for the first received message of the room until user taps it.

ezze commented 1 year ago

Forgot to say that I tested on two different Xiaomi Redmi 9 Pro devices with recent OTA-24 and on BQ Aquaris M10 tablet. So, I guess, the issue is not device specific.

nitanmarcel commented 1 year ago

We don't have support yet for opening chats from the notification, but is shouldn't show a white page anyway. Could be something to do with latest QtWebEngine

Anyway, it's not a breaking bug, I've temporarly moved to other projects until focal gets releasedso I'll fix the issue there and also update the cinny source

ezze commented 1 year ago

@nitanmarcel Ok, thanks for reply. Let's wait for focal. Fixing white screen at least would be great!