telegramdesktop / tdesktop

Telegram Desktop messaging app
https://desktop.telegram.org/
Other
26.2k stars 5.2k forks source link

Can't select multiple service messages to delete them (can do one by one) #26763

Open php4fan opened 1 year ago

php4fan commented 1 year ago

Steps to reproduce

In a group, as an administrator, I can delete each and every one of these service messages:

image

Expected behaviour

I should be able to select as many as I want of them by clicking and dragging, just like I can with regular messages.

If I am allowed to do any action on any of them one by one, I have to be able to select a few to apply the same action to all of them.

Actual behaviour

I can right click individually and delete each and every one of them one by one; however, I cannot select more than one (or actually, even one).

Operating system

Manjaro Linux

Version of Telegram Desktop

4.8.4

Installation source

Other (unofficial) source

Crash ID

No response

Logs

No response

ilya-fedin commented 1 year ago

This is something I lack as well but I bet it's not a bug

php4fan commented 1 year ago

Well it's a feature that is necessary, that is missing, and whose absence is completely stupid and has a huge impact. Call it what you want, it's something that needs fixing. If it's a design decision, it's a wrong one that has no possible justification. But I bet it's just an oversight. To me the distinction between bug and design flaw is pointless unless it's useful to decide who needs to fix it.

ilya-fedin commented 1 year ago

Well, if this is made intentionally then this either should be closed or reported as a feature request

php4fan commented 1 year ago

Unless you know someone to whom you can ask if this was intentional, I would assume it wasn't. I cannot imagine that the application was designed by such an idiot.

And if that was actually the case, then yes it would make sense to label it as a feature request, but not to close it, unless somebody can show that it's a bad idea for some reason.

john-preston commented 1 year ago

It is intentional, only text selection is supported in service messages.

This is done the same way it works on the official mobile apps (iOS and Android), there is no multiselection of service messages as well. Historically this is because they can't be forwarded.

I understand a use-case for mass-deletion for admins with "Delete messages" admin right, but it should work for them only.

This can be left as a enhancement request.

php4fan commented 1 year ago

I understand a use-case for mass-deletion for admins with "Delete messages" admin right, but it should work for them only.

Well isn't it already only admins who can delete them in the first place?

The logic that determines the ability to select shouldn't be based (directly) on what rights you have. What rights you have determines what actions you can perform on items. What actions you can perform on them determine if you need to be able to select them. If there's at least one action that you can perform on each item (whether that's because you have some admin rights or because that's an action that anybody can perform) that it also makes sense to apply to more than one at once (e.g. delete as opposed to reply), then there needs to be the possibility to select them.

Actually, scratch that. What if I just want to select all of them to copy as text and paste? I can't even do that!! Multiple selection has to be available always, even if only to copy as text.