officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu | 18 +++ sw/inc/cmdid.h | 6 - sw/sdi/_tabsh.sdi | 16 +++ sw/sdi/swriter.sdi | 50 ++++++++++ sw/source/uibase/shells/tabsh.cxx | 14 ++ 5 files changed, 100 insertions(+), 4 deletions(-)
New commits: commit 55bcba1e0aba237a873ee1f788761045a8eff846 Author: Philippe Jung <phil.j...@free.fr> Date: Fri Jun 5 11:46:34 2015 +0200 WRITER: readd InsertRows/InsertCols Following ESC decision, uno commands InsertRows / InsertColumns must be added to the current one InsertRowsBefore / InsertColumnsBefore and InsertRowsAfter / InsertColumnsAfter. This will guarantee ascending compatibility while making more explicit the behaviour thanks to the new commands. Change-Id: I31964e3bdfa63b9447edb794fe233e2665651142 Reviewed-on: https://gerrit.libreoffice.org/16097 Reviewed-by: Philippe Jung <phil.j...@free.fr> Tested-by: Jenkins <c...@libreoffice.org> diff --git a/officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu b/officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu index 43e97fa..b72659c 100644 --- a/officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu +++ b/officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu @@ -1144,6 +1144,15 @@ <value>1</value> </prop> </node> + <!-- 2015/06 The next one is to be deprecated one day or another --> + <node oor:name=".uno:InsertRows" oor:op="replace"> + <prop oor:name="Label" oor:type="xs:string"> + <value xml:lang="en-US">Insert Row Below</value> + </prop> + <prop oor:name="Properties" oor:type="xs:int"> + <value>1</value> + </prop> + </node> <node oor:name=".uno:InsertRowsAfter" oor:op="replace"> <prop oor:name="Label" oor:type="xs:string"> <value xml:lang="en-US">Insert Rows Below</value> @@ -1174,6 +1183,15 @@ <value>1</value> </prop> </node> + <!-- 2015/06 The next one is to be deprecated one day or another --> + <node oor:name=".uno:InsertColumns" oor:op="replace"> + <prop oor:name="Label" oor:type="xs:string"> + <value xml:lang="en-US">Insert Column Right</value> + </prop> + <prop oor:name="Properties" oor:type="xs:int"> + <value>1</value> + </prop> + </node> <node oor:name=".uno:InsertColumnsAfter" oor:op="replace"> <prop oor:name="Label" oor:type="xs:string"> <value xml:lang="en-US">Insert Columns Right</value> diff --git a/sw/inc/cmdid.h b/sw/inc/cmdid.h index 7c15b3a..4f77923 100644 --- a/sw/inc/cmdid.h +++ b/sw/inc/cmdid.h @@ -334,20 +334,22 @@ #define FN_TABLE_REP (FN_FORMAT + 99) /* TableRepresentation */ #define FN_CONVERT_TEXT_TABLE (FN_FORMAT + 100) /* convert text <-> table */ -#define FN_TABLE_INSERT_ROW_AFTER (FN_FORMAT + 101) /* */ -#define FN_TABLE_INSERT_COL_AFTER (FN_FORMAT + 102) /* */ +#define FN_TABLE_INSERT_ROW (FN_FORMAT + 101) /* */ +#define FN_TABLE_INSERT_COL (FN_FORMAT + 102) /* */ #define FN_TABLE_DELETE_ROW (FN_FORMAT + 103) /* */ #define FN_TABLE_DELETE_COL (FN_FORMAT + 104) /* */ #define FN_TABLE_SPLIT_CELLS (FN_FORMAT + 105) /* */ #define FN_TABLE_MERGE_CELLS (FN_FORMAT + 106) /* */ #define FN_TABLE_SET_ROW_HEIGHT (FN_FORMAT + 107) /* */ #define FN_TABLE_SET_COL_WIDTH (FN_FORMAT + 108) /* */ +#define FN_TABLE_INSERT_ROW_AFTER (FN_FORMAT + 109) /* */ #define FN_OPTIMIZE_TABLE (FN_FORMAT + 110) /* ToolBoxItem for optimizing tables */ #define FN_TABLE_INSERT_ROW_BEFORE (FN_FORMAT + 111) /* */ #define FN_TABLE_INSERT_COL_BEFORE (FN_FORMAT + 112) /* */ #define FN_TABLE_SELECT_ROW (FN_FORMAT + 113) /* */ #define FN_TABLE_SELECT_COL (FN_FORMAT + 114) /* */ #define FN_TABLE_SELECT_ALL (FN_FORMAT + 115) /* */ +#define FN_TABLE_INSERT_COL_AFTER (FN_FORMAT + 116) /* */ #define FN_TABLE_SET_READ_ONLY_CELLS (FN_FORMAT + 117) /* protect table cells */ #define FN_TABLE_UNSET_READ_ONLY_CELLS (FN_FORMAT + 119) /* undo table cell protection */ #define FN_TABLE_HEADLINE_REPEAT (FN_FORMAT + 120) /* also used in SwXTextTable*/ diff --git a/sw/sdi/_tabsh.sdi b/sw/sdi/_tabsh.sdi index 63792fe..b8be301 100644 --- a/sw/sdi/_tabsh.sdi +++ b/sw/sdi/_tabsh.sdi @@ -148,6 +148,14 @@ interface BaseTextTable DisableFlags="SW_DISABLE_ON_PROTECTED_CURSOR"; ] + // 2015/06 Deprecated, replaced by INSERT_COL_AFTER + FN_TABLE_INSERT_COL // status(final|play) + [ + ExecMethod = Execute ; + StateMethod = NoState ; + DisableFlags="SW_DISABLE_ON_PROTECTED_CURSOR"; + ] + FN_TABLE_INSERT_COL_AFTER // status(final|play) [ ExecMethod = Execute ; @@ -176,6 +184,14 @@ interface BaseTextTable DisableFlags="SW_DISABLE_ON_PROTECTED_CURSOR"; ] + // 2015/06 Deprecated, replaced by INSERT_ROW_AFTER + FN_TABLE_INSERT_ROW // status(final|play) + [ + ExecMethod = Execute ; + StateMethod = GetState ; + DisableFlags="SW_DISABLE_ON_PROTECTED_CURSOR"; + ] + FN_TABLE_INSERT_ROW_DLG // status(final|play) [ Export = FALSE ; diff --git a/sw/sdi/swriter.sdi b/sw/sdi/swriter.sdi index 2368d9d..03b94c1 100644 --- a/sw/sdi/swriter.sdi +++ b/sw/sdi/swriter.sdi @@ -3634,6 +3634,31 @@ SfxVoidItem InsertColumnsBefore FN_TABLE_INSERT_COL_BEFORE GroupId = GID_TABLE; ] +/* 2015/06 Deprecated, replaced by InsertColumsAfter */ +SfxVoidItem InsertColumns FN_TABLE_INSERT_COL +() +[ + /* flags: */ + AutoUpdate = FALSE, + Cachable = Cachable, + FastCall = TRUE, + HasCoreId = FALSE, + HasDialog = TRUE, + ReadOnlyDoc = FALSE, + Toggle = FALSE, + Container = FALSE, + RecordAbsolute = FALSE, + RecordPerSet; + Asynchron; + + /* config: */ + AccelConfig = FALSE, + MenuConfig = FALSE, + StatusBarConfig = FALSE, + ToolBoxConfig = TRUE, + GroupId = GID_TABLE; +] + SfxVoidItem InsertColumnsAfter FN_TABLE_INSERT_COL_AFTER () [ @@ -4404,6 +4429,31 @@ SfxVoidItem InsertRowsBefore FN_TABLE_INSERT_ROW_BEFORE GroupId = GID_TABLE; ] +/* 2015/06 Deprecated, replaced by InsertRowsAfter */ +SfxVoidItem InsertRows FN_TABLE_INSERT_ROW +() +[ + /* flags: */ + AutoUpdate = FALSE, + Cachable = Cachable, + FastCall = TRUE, + HasCoreId = FALSE, + HasDialog = TRUE, + ReadOnlyDoc = FALSE, + Toggle = FALSE, + Container = FALSE, + RecordAbsolute = FALSE, + RecordPerSet; + Asynchron; + + /* config: */ + AccelConfig = FALSE, + MenuConfig = FALSE, + StatusBarConfig = FALSE, + ToolBoxConfig = TRUE, + GroupId = GID_TABLE; +] + SfxVoidItem InsertRowsAfter FN_TABLE_INSERT_ROW_AFTER () [ diff --git a/sw/source/uibase/shells/tabsh.cxx b/sw/source/uibase/shells/tabsh.cxx index b1a6fc0..13aa5df 100644 --- a/sw/source/uibase/shells/tabsh.cxx +++ b/sw/source/uibase/shells/tabsh.cxx @@ -820,13 +820,20 @@ void SwTableShell::Execute(SfxRequest &rReq) OSL_ENSURE( false, "function may not be called now." ); break; + + // 2015/06 The following two are deprecated but kept for ascending + // compatibility + case FN_TABLE_INSERT_COL: + case FN_TABLE_INSERT_ROW: + // fallback case FN_TABLE_INSERT_COL_BEFORE: case FN_TABLE_INSERT_ROW_BEFORE: case FN_TABLE_INSERT_COL_AFTER: case FN_TABLE_INSERT_ROW_AFTER: { bool bColumn = rReq.GetSlot() == FN_TABLE_INSERT_COL_BEFORE - || rReq.GetSlot() == FN_TABLE_INSERT_COL_AFTER; + || rReq.GetSlot() == FN_TABLE_INSERT_COL_AFTER + || rReq.GetSlot() == FN_TABLE_INSERT_COL; sal_uInt16 nCount = 0; bool bAfter = true; if (pItem) @@ -860,7 +867,9 @@ void SwTableShell::Execute(SfxRequest &rReq) nCount = maxY - minY + 1; } bAfter = rReq.GetSlot() == FN_TABLE_INSERT_COL_AFTER - || rReq.GetSlot() == FN_TABLE_INSERT_ROW_AFTER; + || rReq.GetSlot() == FN_TABLE_INSERT_ROW_AFTER + || rReq.GetSlot() == FN_TABLE_INSERT_ROW + || rReq.GetSlot() == FN_TABLE_INSERT_COL; } if( nCount ) @@ -1251,6 +1260,7 @@ void SwTableShell::GetState(SfxItemSet &rSet) } break; } + case FN_TABLE_INSERT_ROW: case FN_TABLE_INSERT_ROW_AFTER: case FN_TABLE_INSERT_ROW_DLG: if ( rSh.IsInRepeatedHeadline() ) _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits