Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"unread messages" counter on menu icon for non-text-events but nothing shown in unread messages tab or spaces list. #110

Closed
dreirund opened this issue Jan 26, 2022 · 16 comments
Labels
question Further information is requested

Comments

@dreirund
Copy link

Describe the bug
SchildiChat-Android shows an "unread messages" counter on the menu icon, but on the tab for unread notifications and also in the list of my spaces I don't see them.

To get this counter removed, I manually have to open each room in order to find the one which has caused this; if I open the correct room it get's removed.

Messages which are text from a user show up in the "unread notifications" tab; and the messages which trigger this notifications which do not show up are, I think, stuff like message deletion or avatar change or other meta information.

It goes away if I switch off
"Menu" → "Settings" → "Preferences" → "Count muted messages",
however, since I have not muted any room or user, I would not expect that this would make any influence here.

I have set
"Menu" → "Settings" → "Labs" → "Add a dedicated tab for unread notifications on main screen."
to on.

Smartphone (please complete the following information):

  • Device: markw (Xiaomi Redmi 4 Prime)
  • OS: crDroid 11.0-20210522 v7.6 (Android 11)

Additional context

  • App version: 1.3.8.sc47 [40100720] (F-d3ac345c)
  • Store: F-Droid

Upstream relevance

  • Does this issue exist in Element Android?
    I have not noticed so far.

Screenshots:
SchildiChat unread messages tab SchildiChat spaces list

@SpiritCroc
Copy link
Member

The "notifications" tab, by design, does not include silent messages (such as from muted chats, but also other events such as bot messages). Element does not counted muted messages at all, so their behavior should match the "Count muted messages" setting turned to off.

however, since I have not muted any room or user, I would not expect that this would make any influence here.

The unread messages counter may also count chats that are marked as unread without any notifications e.g. due to "silent" events, such as bot notices.

I think, for your scenario, it would be sufficient to disable "Count muted messages", wouldn't it?
Or would you prefer a setting to also include mute unreads in the notifications tab, or even another "unread" tab?

@SpiritCroc SpiritCroc added the question Further information is requested label Jan 27, 2022
@dreirund
Copy link
Author

dreirund commented Jan 28, 2022 via email

@SpiritCroc
Copy link
Member

So "Count muted messages" does exactly turn off the counter on the menu button, but does not change anything else?

It basically hides all the unread counters with gray background, and keeps the green and red ones.

But, what I complain about here is that if muted messages are counted, it is not easy to find out where they did occur. I would expect that then the counters are also on the spaces and rooms where the muted messages did appear. Regards!

That sounds like a bug, these unread counters should show for spaces (and they do for me).
For rooms, these counts show if your homeserver actually reports unread message counts for chats (which current versions of synapse do, but I don't know what homeserver you use) - but, if the room shows as unread (i.e. there is a colored bar on the left of the chat in the overview) without any server-reported count, the counting algorithm just assumes there is "1" unread message in there. Sometimes, if the app hasn't fetched enough history of some rooms to know whether you actually read the latest message, it will also show it as unread. In this case, it helps to just open those chats manuall, and afterwards, unread counts should be and remain accurate.

@dreirund
Copy link
Author

dreirund commented Jan 29, 2022 via email

@SpiritCroc
Copy link
Member

For me not; tell me if there is anything I can help to track this down.

What homeserver are you using?

All that does not help if no room is marked where the counted "muted" message did occur. (And, if I open a chat, the unread count does go away anyway.) For me it makes no sense if those messages are counted globally but there is no hint where that count came from (either as an accurate count on the space and room or just a hint on the space and room that something is there). For not-muted messages all that works.

So, just to confirm I understand this issue correctly:

  • the total unread counter on the top left of the top bar shows a count > 0 for unread notifications, and
  • none of your chats show up as unread (i.e. there is no colored bar on the left of any of your chats in the list (any space or no space, really check all your chats))

Is that right?

You say there is no counter in the spaces list, does this also mean that there is no counter for the "home" entry above the spaces list?

Since you're running a slightly outdated app version (according to your initial report), it makes also sense to test again on 1.3.12.*, just to make sure.

@dreirund
Copy link
Author

dreirund commented Jan 31, 2022 via email

@SpiritCroc
Copy link
Member

I would rather not tell publicly because of privacy issues (it is not matrix.org). Can I tell you somehow privately?

Server software (synapse, conduit, dendrite?) and version would be all I'm interested in in this case.

I use what ever is up to date on F-Droid. Sorry, I won't use Google Play Store, and I also can't manually install an .apk without having to uninstall the F-Droid app first, which would invalidate my settings (also settings I did for that app in other apps, like Autostarts and X Privacy Lua). So, I can only test what is newest on F-Droid.

Not using Google Play is totally fine. I don't see why you would need to uninstall F-Droid for installing an apk manually. In case you didn't know, I'm also providing my own F-Droid repos, which have SchildiChats with different package names, so you can easily install them next to the one in the official F-Droid repo, without having to uninstall that one. More info here: https://schildi.chat/android/

@dreirund
Copy link
Author

dreirund commented Feb 1, 2022 via email

@SpiritCroc
Copy link
Member

An admin of the server told me: Synapse 1.51

Sounds fine.
I guess I'll drop you a custom apk or implement some extra debugging switch to get some logs where the unread counts come from.

@SpiritCroc
Copy link
Member

Hi, sorry for not coming back to you earlier. I'm planning to add a couple more debugging settings to the app later. Do you still have the issue on 1.4.4-based beta releases?

@github-actions
Copy link

This issue is stale because it has been open 150 days with no activity. Remove stale label or comment or this will be closed in 14 days.

@github-actions github-actions bot added the Stale label Dec 24, 2022
@dreirund
Copy link
Author

dreirund commented Dec 24, 2022 via email

@SpiritCroc
Copy link
Member

SpiritCroc commented Dec 24, 2022

I don't keep track of all issues, since many concern the upstream codebase, and can thus be fixed without me noticing. So it makes sense to me to not keep them open forever, which can lead to confusion.

You didn't even reply to my last question, so how am I supposed to know if this is still an issue in that case. 5 month sounds sensible to me to close stale issues, or require users to report it's still an issue. Seems to me like sensible user-side effort to make my life easier. Note we also exempt some issue labels from automatic staling, so I can add one of those if it makes sense to me.

@dreirund
Copy link
Author

dreirund commented Dec 24, 2022 via email

@github-actions github-actions bot removed the Stale label Dec 25, 2022
@SpiritCroc
Copy link
Member

SpiritCroc commented Dec 25, 2022

The screenshot looks like intended to me. As you can see in the chats screenshot, there is a grey "unread" bar at the left of the chat, which means that the app found some muted message content in there that hasn't been read yet (such as bot notices), or it didn't fetch enough timeline to ensure everything is read. If you want to have the actual unread number there, please make sure your homeserver supports MSC2654 (e.g. in synapse, you can add msc2654_enabled: true to the experimental_features section to do that). If you do not like this feature, disable "Count muted messages" in settings.

So sounds to me like the original issue as of following description no longer exists and this can be closed?

  • the total unread counter on the top left of the top bar shows a count > 0 for unread notifications, and

  • none of your chats show up as unread (i.e. there is no colored bar on the left of any of your chats in the list (any space or no space, really check all your chats))

@dreirund
Copy link
Author

So sounds to me like the original issue as of following description no longer exists and this can be closed?

  • the total unread counter on the top left of the top bar shows a count > 0 for unread notifications, and
  • none of your chats show up as unread (i.e. there is no colored bar on the left of any of your chats in the list (any space or no space, really check all your chats))

Yes.

So I am closing now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants