Bug 458911

Summary: Add ability to enter (optional) deletion reason
Product: [Applications] NeoChat Reporter: Justin Zobel <justin.zobel>
Component: GeneralAssignee: Tobias Fella <fella>
Status: RESOLVED FIXED    
Severity: wishlist CC: carl, janbidler00
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Justin Zobel 2022-09-09 07:35:17 UTC
Add the ability to add a message if desired when deleting messages.

Maybe `Remove (with reason)` as a context menu option.
Comment 1 Jan Bidler 2022-09-11 11:37:49 UTC
Sounds similar to BUG: 457473 for me.

I don't think that another context menu is worth it, just prompting an additional optional text box during delete/kicking/banning etc.
Comment 2 Justin Zobel 2022-09-11 11:55:48 UTC
Provided it's an optional tickbox to enter the message that's fine.

On 11/9/22 21:07, Jan Bidler wrote:
> https://bugs.kde.org/show_bug.cgi?id=458911
>
> Jan Bidler <janbidler00@protonmail.com> changed:
>
>             What    |Removed                     |Added
> ----------------------------------------------------------------------------
>                   CC|                            |janbidler00@protonmail.com
>
> --- Comment #1 from Jan Bidler <janbidler00@protonmail.com> ---
> Sounds similar to BUG: 457473 for me.
>
> I don't think that another context menu is worth it, just prompting an
> additional optional text box during delete/kicking/banning etc.
>
Comment 3 Bug Janitor Service 2022-11-17 23:56:46 UTC
A possibly relevant merge request was started @ https://invent.kde.org/network/neochat/-/merge_requests/636
Comment 4 Tobias Fella 2022-11-18 00:24:49 UTC
Git commit d70f77f6225a7518a81802e8149490168172cb91 by Tobias Fella.
Committed on 17/11/2022 at 23:56.
Pushed by tfella into branch 'master'.

Allow adding a reason for deleting messages

M  +4    -4    src/neochatroom.cpp
M  +2    -2    src/neochatroom.h
M  +6    -2    src/qml/Dialog/UserDetailDialog.qml
M  +4    -4    src/qml/Menu/Timeline/FileDelegateContextMenu.qml
M  +4    -1    src/qml/Menu/Timeline/MessageDelegateContextMenu.qml
A  +53   -0    src/qml/Menu/Timeline/RemoveSheet.qml     [License: GPL(v2.0+)]
M  +1    -0    src/res.qrc

https://invent.kde.org/network/neochat/commit/d70f77f6225a7518a81802e8149490168172cb91