sw/sdi/_grfsh.sdi |    1 -
 1 file changed, 1 deletion(-)

New commits:
commit c769c369c87a46ec877b7eefee27988044a2798f
Author:     Miklos Vajna <vmik...@collabora.com>
AuthorDate: Fri Jul 1 11:40:10 2022 +0200
Commit:     Miklos Vajna <vmik...@collabora.com>
CommitDate: Fri Jul 1 13:08:55 2022 +0200

    sw content controls: fix picture placeholders
    
    This went wrong in commit c321498f915f4e8b3f4853232860ce040ab48e46 (sw
    content controls: reject typing inside checkbox or picture content
    controls, 2022-06-10), now a freshly inserted picture content control is
    not replaced with an image on click.
    
    The problem is that we want to forbid typing into a picture content
    control (it should only host a single as-char image), but changing the
    picture's bitmap is meant to be still possible.
    
    Fix the problem by allowing SID_CHANGE_PICTURE even if the cursor is
    protected: this allows changing the picture again, and a real read-only
    document still doesn't show the "change" menu item in its context menu.
    
    If we later find some other corner-case where this command should be
    still disabled, we can extend SwGrfShell::GetAttrState(), similar to how
    e.g. inserting images in input fields is disabled there.
    
    Change-Id: I8b55e930330b9748ecda950dedae907b86c57e2f
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136729
    Reviewed-by: Miklos Vajna <vmik...@collabora.com>
    Tested-by: Jenkins

diff --git a/sw/sdi/_grfsh.sdi b/sw/sdi/_grfsh.sdi
index 8291745966bb..5475017d448e 100644
--- a/sw/sdi/_grfsh.sdi
+++ b/sw/sdi/_grfsh.sdi
@@ -63,7 +63,6 @@ interface BaseTextGraphic
     [
         ExecMethod = Execute ;
         StateMethod = GetAttrState ;
-        DisableFlags="SfxDisableFlags::SwOnProtectedCursor";
     ]
 
     SID_EXTERNAL_EDIT

Reply via email to