[Bug 160219] Add confirmation dialog to "Delete all comments" (.uno:DeleteAllNotes) in Calc

[bugzilla-daemon at bugs.documentfoundation.org]

https://bugs.documentfoundation.org/show_bug.cgi?id=160219

Heiko Tietze <heiko.tietze at documentfoundation.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|needsUXEval                 |difficultyBeginner,
                   |                            |easyHack, skillCpp, topicUI
                 CC|libreoffice-ux-advise at lists |heiko.tietze at documentfounda
                   |.freedesktop.org            |tion.org,
                   |                            |mentoring at documentfoundatio
                   |                            |n.org,
                   |                            |michael.stahl at allotropia.de

--- Comment #4 from Heiko Tietze <heiko.tietze at documentfoundation.org> ---
Support from my side for the confirmation, mentioned it recently in bug 160598
comment 7. Since the function can be undone, at least in Writer, it's not super
important and we could add an option to not ask again. But I believe Delete-All
is not a frequently used function that we have to make usable as easy as
possible.

Code pointer:

Calc
* sc/source/ui/view/cellsh1.cxx > ExecuteEdit() > FID_DELETE_ALL_NOTES

Writer
* sw/source/uibase/shells/textfld.cxx > ExecField() > FN_DELETE_ALL_NOTES
* sw/source/uibase/shells/annotsh.cxx > NoteExec() > FN_DELETE_ALL_NOTES
Don't know why this command has two implementations here and perhaps we add the
confirmation in the postitmgr class at
* sw/source/uibase/docvw/PostItMgr.cxx > SwPostItMgr::Delete()

Draw/Impress
* sd/source/ui/annotations/annotationmanager.cxx > ExecuteDeleteAnnotation() >
SID_DELETEALL_POSTIT

-- 
You are receiving this mail because:
You are on the CC list for the bug.